Dan Burkert has submitted this change and it was merged. Change subject: KUDU-1792 It is possible to overwrite range partitions ......................................................................
KUDU-1792 It is possible to overwrite range partitions This fixes another bug (see KUDU-1750) in the master code which detects overlaps in range partitions during alter table operations. A new test is included which attempts to exhaustively cover all cases of conflict between pairs of ranges being added. It also adds a lot of coverage for drop conflicts between pairs of ranges, although the coverage is not intended to be exhaustive. The next step if we want to test this space even more would be to increase the number of ranges involved in conflicts to three or more, but since this amplifies the search space so much, it is probably best done as part of a randomized fuzz test. Change-Id: I89ae162f33de20169848091e518315853be30460 Reviewed-on: http://gerrit.cloudera.org:8080/5395 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins --- M python/kudu/tests/test_client.py M src/kudu/integration-tests/alter_table-test.cc M src/kudu/master/catalog_manager.cc 3 files changed, 329 insertions(+), 4 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/5395 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I89ae162f33de20169848091e518315853be30460 Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Dan Burkert <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <[email protected]>
