accumulo icon indicating copy to clipboard operation
accumulo copied to clipboard

checks the entire map, solves #3522

Open rsingh433 opened this issue 1 year ago • 3 comments

New changes: requireSame method in the ConditionalTabletMutator interface, aiming to ensure that tablet metadata for specified columns matches a given TabletMetadata object. The method utilizes a static compareMaps method from the Ample class to compare maps and throws a RuntimeException if the tablet files are not the same. The goal is to ensure that the tablet files (represented by a map) in the provided TabletMetadata object are the same as the tablet files in the current context. If the maps are not equal, the RuntimeException is thrown, indicating that the tablet files are not the same.

rsingh433 avatar Jan 19 '24 20:01 rsingh433

I don't think this is the correct interpretation of #3522. I think that issue is saying to modify the code here when looking at the files.

dlmarion avatar Jan 22 '24 19:01 dlmarion

I don't think this is the correct interpretation of #3522. I think that issue is saying to modify the code here when looking at the files.

Thank you for guiding me in the right direction. @keith-turner, could you please verify if the modifications I implemented in the code align with your expectations?

rsingh433 avatar Jan 25 '24 00:01 rsingh433

Now that #4227 is merged, it would be good to merge elasticity into this branch and then add unit test for this new code.

keith-turner avatar Feb 15 '24 16:02 keith-turner

Superceded by #4486

dlmarion avatar May 10 '24 13:05 dlmarion