akashrn5 opened a new pull request #3907:
URL: https://github.com/apache/carbondata/pull/3907


    ### Why is this PR needed?
   During the carbondata reliability and concurrency test of load, compaction 
and query, Some times nullPointerException is thrown. This is because, in 
`TableSegmentRefresher ` we get the last modified timestamp of the segment 
file, to decide to refresh the cache, in case of concurrency, it can happen 
that the segment file get deleted or during update, file may not be there, that 
time getLastModified time throws null pointer.
    
    ### What changes were proposed in this PR?
   Before get the last modified time, always check for the file exists, as it 
can be deleted during that time due to concurrency, if not present, initialize 
to zero.
       
    ### Does this PR introduce any user interface change?
    - No
   
    ### Is any new testcase added?
    - No(verified with concurrency of 1000s of segments.
   
       
   


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to