[GitHub] [accumulo] ctubbsii commented on issue #1893: Improve handling of interrupt conditions.

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

ctubbsii commented on issue #1893:
URL: https://github.com/apache/accumulo/issues/1893#issuecomment-771031662


   > Some parts of the code could use this util:
   > https://github.com/apache/accumulo/blob/3fd5cad92f9b63ac19e4466f3f2d5237b905262c/core/src/main/java/org/apache/accumulo/fate/util/UtilWaitThread.java#L49
   > 
   > 
   > But it should probably be moved out of the FATE package. It is already used in a bunch of ITs.
   
   This is a holdover from when accumulo-fate was a separate jar. A lot of the fate stuff was left in their current packages to avoid making any changes that might affect FaTE serialization. This one is probably safe to move, since it is a static void method only, and there's probably other generic utilities that can be moved as well, but since it's all internal, it doesn't matter that much to move it. But, if somebody does move it, they should be careful to avoid breaking FaTE serialization by moving serialized objects.


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