MartijnVisser commented on PR #19428:
URL: https://github.com/apache/flink/pull/19428#issuecomment-1098820075

   @David-N-Perkins Thanks a lot for the fix! I've squashed the commits and 
rebased the PR. I've also modified the commit message to be in line with Flink 
code contribution guide, see 
https://flink.apache.org/contributing/contribute-code.html
   
   Since this involves S3, I need to run some manually tests before I can merge 
the PR. I'll try to get that done today, finishtthe review and then I'll merge 
it (if everything is OK of course).
   
   Thanks again for your help. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to