zentol opened a new pull request #7261: [FLINK-11085][s3] Fix inclusion of 
presto hadoop classes
URL: https://github.com/apache/flink/pull/7261
 
 
   ## What is the purpose of the change
   
   This PR fixes the bundling of hadoop classes from 
`com.facebook.presto.hadoop:hadoop-apache2` in the presto-s3 filesystem module.
   
   As of right now I do not really know why this fixes the issue, but intend to 
file an issue with MSHADE.
   
   ## Brief change log
   
   * reorder filter definitions
   
   ## Verifying this change
   
   Manually verified. Ensure that 
`org\apache\flink\fs\s3presto\shaded\com\facebook\presto\hadoop\` is contained 
in the jar, and no other contents from this jar are included (for example, 
`nativelib`).
   
   Optionally you can also run the presto e2e test if you also include the 
commits from https://github.com/zentol/flink/tree/11100.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to