[GitHub] flink issue #6302: [FLINK-9061][checkpointing] add entropy to s3 path for be...

2018-07-20 Thread indrc
Github user indrc commented on the issue: https://github.com/apache/flink/pull/6302 @mdaxini i don't think we have to validate the public api; with the 4 char alphanumeric there is more than enough entropy to cover all filewrites in a conceivable TM cluster. And to add to that,

[GitHub] flink issue #6302: [FLINK-9061][checkpointing] add entropy to s3 path for be...

2018-07-17 Thread indrc
Github user indrc commented on the issue: https://github.com/apache/flink/pull/6302 @StefanRRichter @StephanEwen okay, added the suggested changes; let me know. ---

[GitHub] flink issue #6302: [FLINK-9061][checkpointing] add entropy to s3 path for be...

2018-07-15 Thread indrc
Github user indrc commented on the issue: https://github.com/apache/flink/pull/6302 @StephanEwen Thanks for your comments and feedback, my replies below 1. after talking to @StefanRRichter we decided to bring in this dependency to use a standard library instead; initially the pr

[GitHub] flink pull request #6302: [FLINK-9061][checkpointing] add entropy to s3 path...

2018-07-10 Thread indrc
GitHub user indrc opened a pull request: https://github.com/apache/flink/pull/6302 [FLINK-9061][checkpointing] add entropy to s3 path for better scalability ## What is the purpose of the change *This pull request adds entropy to the checkpoint path based upon a user