Commit c92e4614 authored by marko's avatar marko

branches/zip: page_cur_delete_rec(): Remove the bogus page_zip_validate_low()

assertion that was added in r2781 and explain why it was bogus.
parent 626f02fb
...@@ -1766,11 +1766,15 @@ page_cur_delete_rec( ...@@ -1766,11 +1766,15 @@ page_cur_delete_rec(
page = page_cur_get_page(cursor); page = page_cur_get_page(cursor);
page_zip = page_cur_get_page_zip(cursor); page_zip = page_cur_get_page_zip(cursor);
#ifdef UNIV_ZIP_DEBUG
/* Strict page_zip_validate() may fail here when /* page_zip_validate() will fail here when
btr_cur_pessimistic_delete() invokes btr_set_min_rec_mark(). */ btr_cur_pessimistic_delete() invokes btr_set_min_rec_mark().
ut_a(!page_zip || page_zip_validate_low(page_zip, page, TRUE)); Then, both "page_zip" and "page" would have the min-rec-mark
#endif /* UNIV_ZIP_DEBUG */ set on the smallest user record, but "page" would additionally
have it set on the smallest-but-one record. Because sloppy
page_zip_validate_low() only ignores min-rec-flag differences
in the smallest user record, it cannot be used here either. */
current_rec = cursor->rec; current_rec = cursor->rec;
ut_ad(rec_offs_validate(current_rec, index, offsets)); ut_ad(rec_offs_validate(current_rec, index, offsets));
ut_ad(!!page_is_comp(page) == dict_table_is_comp(index->table)); ut_ad(!!page_is_comp(page) == dict_table_is_comp(index->table));
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment