nicknezis commented on PR #3838: URL: https://github.com/apache/incubator-heron/pull/3838#issuecomment-1140626226
> The commit looks good. Just wonder what was the original reason of using a random name in bookkeeper upload? It looks like a bit of code added early in this [PR](https://github.com/apache/incubator-heron/pull/601). Someone suggested they use a timestamp instead of random to see which package was newest. But it doesn't seem to have much thought put behind the original Random Long. I'll have to look further back in the history to determine when it was added. -- 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]
