[GitHub] [hbase] virajjasani commented on issue #1507: HBASE-24151 The master server aborted for IllegalThreadStateException

2020-04-15 Thread GitBox 
virajjasani commented on issue #1507: HBASE-24151 The master server aborted for 
IllegalThreadStateException
URL: https://github.com/apache/hbase/pull/1507#issuecomment-613910761
 
 
   Btw @G302 let's use same logic as master. i.e.
   ```
   // avoid infinite nesting
   if (getClass().isAssignableFrom(balancerClass)) {
 balancerClass = LoadBalancerFactory.getDefaultLoadBalancerClass();
   }
   ```


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:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] [hbase] virajjasani commented on issue #1507: HBASE-24151 The master server aborted for IllegalThreadStateException

2020-04-15 Thread GitBox 
virajjasani commented on issue #1507: HBASE-24151 The master server aborted for 
IllegalThreadStateException
URL: https://github.com/apache/hbase/pull/1507#issuecomment-613868780
 
 
   > @virajjasani
   > Thanks for your reply. I think this problem has been conveniently fixed in 
the master branch. (HBASE-22971)
   
   Got it. Thanks, will backport this to all branch-2s.


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:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] [hbase] virajjasani commented on issue #1507: HBASE-24151 The master server aborted for IllegalThreadStateException

2020-04-14 Thread GitBox 
virajjasani commented on issue #1507: HBASE-24151 The master server aborted for 
IllegalThreadStateException
URL: https://github.com/apache/hbase/pull/1507#issuecomment-613508043
 
 
   Thanks @G302 
   Can you create PR for master? We can let QA run once and I will commit this 
to master and all branch-2s together.


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:
us...@infra.apache.org


With regards,
Apache Git Services