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

Posted to dev@storm.apache.org by GitBox <gi...@apache.org> on 2020/12/04 17:50:02 UTC

[GitHub] [storm] agresch opened a new pull request #3357: STORM-3720 get correct results for BlobStoreFile getModTime()

agresch opened a new pull request #3357:
URL: https://github.com/apache/storm/pull/3357


   ## What is the purpose of the change
   
   If we call getModTime() multiple times before and after a blobstore file changes, we will get an incorrect timestamp.
   
   ## How was the change tested
   
   Built storm.  Ran HdfsBlobStoreImplTest.


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

[GitHub] [storm] agresch merged pull request #3357: STORM-3720 get correct results for BlobStoreFile getModTime()

Posted by GitBox <gi...@apache.org>.

agresch merged pull request #3357:
URL: https://github.com/apache/storm/pull/3357


   


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

[GitHub] [storm] agresch commented on pull request #3357: STORM-3720 get correct results for BlobStoreFile getModTime()

Posted by GitBox <gi...@apache.org>.

agresch commented on pull request #3357:
URL: https://github.com/apache/storm/pull/3357#issuecomment-742582582


   I did not see any issue given how we are using it.  But if the code changed, it seems like we could get unexpected results from calling this multiple times.  


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