Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BTreeIndex Charge Bytes Limit #1489

Merged

Conversation

kunga
Copy link
Member

@kunga kunga commented Jan 31, 2024

Changelog entry

...

Changelog category

  • Not for changelog (changelog entry is not required)

Additional information

Support bytes limit in TChargeBTreeIndex: stop iteration when we touched enough data pages.

Do not apply bytes limit on the first touched page because begin row or first key position may vary.

Limit each additional column family separately to simplify the process.

Part of KIKIMR-19522, #1483

Copy link

github-actions bot commented Jan 31, 2024

2024-01-31 16:27:58 UTC Pre-commit check for 18aae32 has started.
2024-01-31 16:27:59 UTC Build linux-x86_64-release-asan is running...
🟢 2024-01-31 16:43:07 UTC Build successful.
2024-01-31 16:43:17 UTC Tests are running...
🔴 2024-01-31 18:46:02 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
16030 15917 0 17 58 38

Copy link

github-actions bot commented Jan 31, 2024

2024-01-31 16:31:11 UTC Pre-commit check for 18aae32 has started.
2024-01-31 16:31:12 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-01-31 16:44:08 UTC Build successful.
2024-01-31 16:44:17 UTC Tests are running...
🔴 2024-01-31 18:38:32 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
60353 51048 0 1 9270 34

@kunga kunga force-pushed the KIKIMR-19522-btree-precharge-bytes-limit branch from 0ddac53 to 02a2c61 Compare February 1, 2024 18:49
Copy link

github-actions bot commented Feb 1, 2024

2024-02-01 18:52:33 UTC Pre-commit check for 42d9c22 has started.
2024-02-01 18:52:35 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-02-01 18:54:17 UTC Build successful.
2024-02-01 18:54:33 UTC Tests are running...
🟢 2024-02-01 20:09:11 UTC Tests successful.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
58834 49536 0 0 9261 37

Copy link

github-actions bot commented Feb 1, 2024

2024-02-01 18:52:36 UTC Pre-commit check for 42d9c22 has started.
2024-02-01 18:52:39 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-01 18:54:23 UTC Build successful.
2024-02-01 18:54:37 UTC Tests are running...
🔴 2024-02-01 20:24:26 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14518 14389 0 33 64 32

Copy link

github-actions bot commented Feb 2, 2024

2024-02-02 09:38:43 UTC Pre-commit check for 4626ebd has started.
2024-02-02 09:38:46 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-02-02 09:40:27 UTC Build successful.
2024-02-02 09:40:39 UTC Tests are running...
🔴 2024-02-02 11:13:23 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
58837 49534 0 2 9263 38

Copy link

github-actions bot commented Feb 2, 2024

2024-02-02 09:38:54 UTC Pre-commit check for 4626ebd has started.
2024-02-02 09:38:57 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-02 09:40:45 UTC Build successful.
2024-02-02 09:40:57 UTC Tests are running...
🔴 2024-02-02 11:18:42 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14520 14386 0 29 69 36

@kunga kunga marked this pull request as ready for review February 2, 2024 09:43
@kunga kunga requested a review from snaury February 2, 2024 09:43
@kunga kunga mentioned this pull request Feb 2, 2024
62 tasks
@CyberROFL
Copy link
Member

dq tests broken by #1503

@kunga kunga merged commit 5f6a40a into ydb-platform:main Feb 2, 2024
2 of 4 checks passed
@starlinskiy starlinskiy mentioned this pull request Feb 12, 2024
@vitstn vitstn mentioned this pull request Feb 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants