You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2020/10/09 14:18:34 UTC

[GitHub] [flink] StephanEwen edited a comment on pull request #13538: [FLINK-19498][Connector Files] Port LocatableInputSplitAssigner to new File Source API

StephanEwen edited a comment on pull request #13538:
URL: https://github.com/apache/flink/pull/13538#issuecomment-706104404


   ~I would then merge this PR and open a follow-up PR to make sure we have consistent hostnames.~
   
   Turns out this is a very simple addition that fits very well directly in the `LocalityAwareSplitAssigner`, so I'll just fix this on top of this code during merge.
   
   That also has the advantage that we don't drop FQDN information in the source interface in general, in case other sources (non file sources) would actually need them.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org