[GitHub] [flink] yanyan300300 edited a comment on issue #8215: [FLINK-8801][yarn/s3] Fix jars downloading issues due to inconsistent timestamp in S3 Filesystem

[GitBox <gi...@apache.org>]

yanyan300300 edited a comment on issue #8215: [FLINK-8801][yarn/s3] Fix jars downloading issues due to inconsistent timestamp in S3 Filesystem
URL: https://github.com/apache/flink/pull/8215#issuecomment-488062731
 
 
   > Indeed, it looks like this is the better approach. Aside from some minor details in the code, I would only have one concern which I'm not sure about:
   > 
   > * Are we allowed to change the local resource's time? What are the implications?
   > 
   > I guess, this is the local resource file that was uploaded to the JM and there are no further implications but I'm not 100% sure.
   
   Yes, Flink will need to set the local resource's time explicitly. This is also the case before your change.

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


With regards,
Apache Git Services