qiuchenjian commented on issue #3130: [CARBONDATA-3297] Fix that the 
IndexoutOfBoundsException when creating table and dropping table are at the 
same time
URL: https://github.com/apache/carbondata/pull/3130#issuecomment-471416326
 
 
   > > > @qiuchenjian i think red lock should be added for `listAllTables` APi 
also, please check it once
   > > 
   > > 
   > > @akashrn5
   > > done!
   > > I also clone a new seq, besause it's useless that return the same 
ArrayBuffer
   > > Please review and merge, thank you
   > 
   > @qiuchenjian sorry i did not get this `besause it's useless that return 
the same ArrayBuffer`
   
   @akashrn5 
   I mean that if "listAllTables" return "carbonTables" directly and the 
readLock is finished, "carbonTables" will be traversed without protection of 
lock 
   so i return a clone of "carbonTables", it's thread-safe

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to