NihalJain commented on PR #5594:
URL: https://github.com/apache/hbase/pull/5594#issuecomment-1865258878

   Logs (with extra debug info) to demonstrate why the tests were failing. It 
can be seen the count was not 1 as it was expected in test assertion. 
   
   Changed logic to do counting relatively instead of absolutely, which was 
causing flakiness, due to test case ordering.
   ```
   server_name=HOST_NAME,63596,1703112668148
   2023-12-21T04:22:20,258 DEBUG [Listener at localhost/63569-EventThread {}] 
zookeeper.ZKWatcher(604): master:63585-0x100004905c30000, 
quorum=127.0.0.1:64405, baseZNode=/hbase Received ZooKeeper Event, 
type=NodeChildrenChanged, state=SyncConnected, path=/hbase/draining
   2023-12-21T04:22:20,258 INFO  
[RpcServer.default.FPBQ.Fifo.handler=2,queue=0,port=63585 {}] 
master.ServerManager(681): Server HOST_NAME,63596,1703112668148 added to 
draining server list.
   Took 0.0068 seconds                                                          
                                                                                
                                                           
2023-12-21T04:22:20,259 DEBUG [zk-event-processor-pool-0 {}] 
zookeeper.ZKUtil(111): master:63585-0x100004905c30000, quorum=127.0.0.1:64405, 
baseZNode=/hbase Set watcher on existing 
znode=/hbase/draining/HOST_NAME,63602,1703112668656
   2023-12-21T04:22:20,259 DEBUG [zk-event-processor-pool-0 {}] 
zookeeper.ZKUtil(111): master:63585-0x100004905c30000, quorum=127.0.0.1:64405, 
baseZNode=/hbase Set watcher on existing 
znode=/hbase/draining/HOST_NAME,63599,1703112668493
   2023-12-21T04:22:20,259 DEBUG [zk-event-processor-pool-0 {}] 
zookeeper.ZKUtil(111): master:63585-0x100004905c30000, quorum=127.0.0.1:64405, 
baseZNode=/hbase Set watcher on existing 
znode=/hbase/draining/HOST_NAME,63596,1703112668148
   2023-12-21T04:22:20,259 WARN  [zk-event-processor-pool-0 {}] 
master.ServerManager(677): Server HOST_NAME,63602,1703112668656 is already in 
the draining server list.Ignoring request to add it again.
   2023-12-21T04:22:20,259 INFO  [zk-event-processor-pool-0 {}] 
master.DrainingServerTracker(92): Draining RS node created, adding to list 
[HOST_NAME,63602,1703112668656]
   2023-12-21T04:22:20,259 WARN  [zk-event-processor-pool-0 {}] 
master.ServerManager(677): Server HOST_NAME,63599,1703112668493 is already in 
the draining server list.Ignoring request to add it again.
   2023-12-21T04:22:20,259 INFO  [zk-event-processor-pool-0 {}] 
master.DrainingServerTracker(92): Draining RS node created, adding to list 
[HOST_NAME,63599,1703112668493]
   2023-12-21T04:22:20,259 WARN  [zk-event-processor-pool-0 {}] 
master.ServerManager(677): Server HOST_NAME,63596,1703112668148 is already in 
the draining server list.Ignoring request to add it again.
   2023-12-21T04:22:20,259 INFO  [zk-event-processor-pool-0 {}] 
master.DrainingServerTracker(92): Draining RS node created, adding to list 
[HOST_NAME,63596,1703112668148]
   DECOMMISSIONED REGION SERVERS
   HOST_NAME,63599,1703112668493
   HOST_NAME,63602,1703112668656
   HOST_NAME,63596,1703112668148
   3 row(s)
   Took 0.0015 seconds
   initial_number_of_rows=3
   2023-12-21T04:22:20,267 DEBUG [Listener at localhost/63569-EventThread {}] 
zookeeper.ZKWatcher(604): master:63585-0x100004905c30000, 
quorum=127.0.0.1:64405, baseZNode=/hbase Received ZooKeeper Event, 
type=NodeDeleted, state=SyncConnected, 
path=/hbase/draining/HOST_NAME,63596,1703112668148
   2023-12-21T04:22:20,267 DEBUG [Listener at localhost/63569-EventThread {}] 
zookeeper.ZKWatcher(604): master:63585-0x100004905c30000, 
quorum=127.0.0.1:64405, baseZNode=/hbase Received ZooKeeper Event, 
type=NodeChildrenChanged, state=SyncConnected, path=/hbase/draining
   2023-12-21T04:22:20,268 INFO  [zk-event-processor-pool-0 {}] 
master.DrainingServerTracker(109): Draining RS node deleted, removing from list 
[HOST_NAME,63596,1703112668148]
   Took 0.0070 seconds                                                          
                                                                                
                                                           
2023-12-21T04:22:20,268 DEBUG [zk-event-processor-pool-0 {}] 
zookeeper.ZKUtil(111): master:63585-0x100004905c30000, quorum=127.0.0.1:64405, 
baseZNode=/hbase Set watcher on existing 
znode=/hbase/draining/HOST_NAME,63602,1703112668656
   2023-12-21T04:22:20,269 DEBUG [zk-event-processor-pool-0 {}] 
zookeeper.ZKUtil(111): master:63585-0x100004905c30000, quorum=127.0.0.1:64405, 
baseZNode=/hbase Set watcher on existing 
znode=/hbase/draining/HOST_NAME,63599,1703112668493
   2023-12-21T04:22:20,269 WARN  [zk-event-processor-pool-0 {}] 
master.ServerManager(677): Server HOST_NAME,63602,1703112668656 is already in 
the draining server list.Ignoring request to add it again.
   2023-12-21T04:22:20,269 INFO  [zk-event-processor-pool-0 {}] 
master.DrainingServerTracker(92): Draining RS node created, adding to list 
[HOST_NAME,63602,1703112668656]
   2023-12-21T04:22:20,269 WARN  [zk-event-processor-pool-0 {}] 
master.ServerManager(677): Server HOST_NAME,63599,1703112668493 is already in 
the draining server list.Ignoring request to add it again.
   2023-12-21T04:22:20,269 INFO  [zk-event-processor-pool-0 {}] 
master.DrainingServerTracker(92): Draining RS node created, adding to list 
[HOST_NAME,63599,1703112668493]
   DECOMMISSIONED REGION SERVERS
   HOST_NAME,63599,1703112668493
   HOST_NAME,63602,1703112668656
   2 row(s)
   Took 0.0025 seconds
   final_number_of_rows=2
   ```


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