keith-turner opened a new issue, #3527:
URL: https://github.com/apache/accumulo/issues/3527

   In #3513 a new class named SelectedFiles was added. This class contains a 
fate transaction id and a set of stored tablet files.  The class represents the 
set of files selected in a tablet for a user initiated compaction.   In #3513 a 
new method named requireSame for checking if something in a tablet is the same 
via conditional mutation was added.  The new requireSame method enables the 
following
   
    1. Read a tablets selected files, files, and compacting files
    2. Add updates to a mutation based on examining the above three sets
    3. Add a condition to the mutation that files, selected files, and 
compacting files must be same
   
   For step 3 above code currently compare the serialized version of the 
SelectedFiles object with what is in the metadata table via a condition on the 
mutation.  Therefore it extremely important that the serializtion is 
deterministic for the same input data and that there are unit test for this.
    
   In [this 
comment](https://github.com/apache/accumulo/pull/3513/files#r1238442161) it was 
suggested that TypeAdapter could be used to achieve serialization that results 
in a comparable string.


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