BsoBird opened a new pull request, #4935:
URL: https://github.com/apache/hive/pull/4935

   ### What changes were proposed in this pull request?
   Modified the modifier of the cache property in the SplitGroup class to make 
it non-static.
   
   
   ### Why are the changes needed?
   Since the Properties object implement HashTable interface, all the methods 
of the HashTable interface are synchronised.
   In a multi-threaded environment, a deadlock will occur when 
propA.equals(propB)  and propB.equals(propA) occur at the same time.
   The problem can be solved with minimal modification by changing 
SplitGroup.cache to a non-static property.
   
   
   ### Does this PR introduce _any_ user-facing change?
   no
   
   ### Is the change a dependency upgrade?
   no
   
   
   ### How was this patch tested?
   no need. Because this patch doesn't change the execution logic of any code
   


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to