keith-turner commented on code in PR #3439:
URL: https://github.com/apache/accumulo/pull/3439#discussion_r1210847240
##########
test/src/main/java/org/apache/accumulo/test/functional/SplitIT.java:
##########
@@ -140,6 +140,8 @@ public void tabletShouldSplit() throws Exception {
VerifyParams params = new VerifyParams(getClientProps(), table, 100_000);
TestIngest.ingest(c, params);
VerifyIngest.verifyIngest(c, params);
+ // ELASTICITY_TODO: Sum of file sizes is zero in
+ // TabletManagementIterator.shouldReturnDueToSplit, so split never
happens
Review Comment:
Tracked down two problems, have submitted a fix for one : #3441
I added a lot of debugging and finally figured out what was happening. I
was seeing the tserver return row `row_00000124%ff;` where `%ff;` is binary.
In the manager it was processing `row_00000124%ef;%bf;%bd;` where
`%ef;%bf;%bd;` is binary. This was caused by data going through String which
#3441 fixes. The row the manager was processing did not actually exists. It
would write a future location for this non-existent tablet and that would cause
the tablet w/o a prev row. I am going to submit another little fix where a
tablet can not write a future if the tablet does not exists, that should not be
able to happen.
--
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]
