DomGarguilo commented on issue #1791: URL: https://github.com/apache/accumulo/issues/1791#issuecomment-763099727
The `assertEquals` that is failing compares the equivalence of the tablets on a server before and after that server is suspended. There is a comment before the assert that reads: > By this point, all tablets should be either hosted or suspended. All suspended tablets should "belong" to the dead tablet servers, and should be in exactly the same place as before any tserver death. I'm not too sure if this statement is true however I have observed tablets travelling between servers before and after being suspended. Below is a printout of the list of tablets on two servers before and after suspension: ```java 2021-01-19T14:27:28,658 [master.SuspendedTabletsIT] DEBUG: Suspended thor:42727: [1;29;28, 1;1<, 1;10;1, 1;11;10, 1;12;11, 1;24;23, 1;4;3, 1;15;14, 1;3;29, 1;27;26, 1;28;27, 1;8;7] 2021-01-19T14:27:28,664 [master.SuspendedTabletsIT] DEBUG: Previously hosted thor:42727: [1;29;28, 1;1<, 1;10;1, 1;11;10, 1;12;11, 1;15;14, 1;3;29, 1;27;26, 1;28;27, 1;8;7] 2021-01-19T14:27:28,664 [master.SuspendedTabletsIT] DEBUG: Suspended thor:44931: [1;21;20, 1;20;2, 1;22;21, 1;23;22, 1;25;24, 1;5;4, 1;26;25, 1;16;15] 2021-01-19T14:27:28,664 [master.SuspendedTabletsIT] DEBUG: Previously hosted thor:44931: [1;21;20, 1;20;2, 1;22;21, 1;23;22, 1;24;23, 1;4;3, 1;25;24, 1;5;4, 1;26;25, 1;16;15] ``` Its a bit hard to read but some tablets that appeared on server A are appearing on server B and vise-versa. This tells me that either the assert in this test is making an incorrect assumption about the nature of tablets on a suspended server, or the test is behaving correctly and catching a valid error. Any thoughts would be helpful. ---------------------------------------------------------------- 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]
