Reamer commented on code in PR #4804:
URL: https://github.com/apache/zeppelin/pull/4804#discussion_r1732365173


##########
zeppelin-interpreter/src/main/java/org/apache/zeppelin/scheduler/NamePrefixThreadFactory.java:
##########
@@ -21,19 +21,19 @@
 import java.util.concurrent.ThreadFactory;
 import java.util.concurrent.atomic.AtomicLong;
 
-public class SchedulerThreadFactory implements ThreadFactory {
+public class NamePrefixThreadFactory implements ThreadFactory {

Review Comment:
   I have renamed the class.
   
   > BTW, it could be reused in many places, do we have a more generic place to 
locate this class?
   
   I couldn't find a good place for the class, so I left it there. If you have 
a suitable place in mind, I can move the class there.



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

Reply via email to