Szehon Ho
Szehon Ho
I did notice it the other day, and also thought it would be a good usability feature (especially as there is snapshot table that keeps the original table). I wonder...
Docs are fine, but to me it would make more sense to add a non-default flag to drop_backup (as mentioned above), if we want to do add a feature here....
Thanks for the changes, will merge tomorrow if no further comments.
Merged, thanks @sririshindra for the changes
I think this is an interesting find, it makes sense to me on the first reading, so partition-spec-id on the header of manifestFile is always the same as partition-spec-id of...
Note: will merge it soon if there's no further comments.
Merged, thanks @ConeyLiu and @rdblue for review.
Yea it looks reasonable. Curious if metadata tables also suffer any problem from this scenario, will need to test when I have time.
Tested the patch with metadata tables, seems to fix all the issues. Also nit: should we add the test to 3.3, so it lives longer (I suppose 3.3 will be...
Merged, thanks @jfz , @nastra @rdblue for review