This ticket is a grab-bag of small changes / improvements we'd like to make before fully GA-ing the S3EmitOperator. Proposed changes are as follows:

1. Convert BucketFileKey from a std::pair to a struct (style guide])
2. remove string validator + import in planner.cpp
3. direct capture of fileBytes in PutObjectAsync callback
4. return after cb in mock PutObjectAsync