VicoWu commented on pull request #1743: URL: https://github.com/apache/hbase/pull/1743#issuecomment-635050379
> @jojochuang > This issue is found by our cluster incident. > But my current commit still has some problem which need to be resolved before finally adopted: > > In this PR, I cache the token between RS and NameNode instead of always applying for a new one every time processing a file, so the token should be confirmed that it is not expired every time we use it; > So I think I need to add a check logic to verify its expiration time , if it is already or going to be expired, I will have to apply for a new one and overwrite the cache; @jojochuang I have added the token expiration processing logic and help to review it; The logic has updated to the PR's description part; ---------------------------------------------------------------- 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
