[GitHub] [hadoop] paulward24 commented on issue #1015: HashMap is not thread safe. Field storageMap is typically synchronized by storageMap. However, in one place, field storageMap is not protected with synchronized.

[GitBox <gi...@apache.org>]

paulward24 commented on issue #1015: HashMap is not thread safe. Field storageMap is typically synchronized by storageMap. However, in one place, field storageMap is not protected with synchronized.
URL: https://github.com/apache/hadoop/pull/1015#issuecomment-505730705
 
 
   I see some tests failed.  I don't know why this small change would break those tests.  Maybe somebody more familiar with the project can take a look.
   
   Either way, the original problem --- using a thread-unsafe HashMap concurrently without synchronization --- remains.

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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org