Skip to content

Fix test_db_counters test#6447

Merged
jepett0 merged 2 commits intoydb-platform:mainfrom
jepett0:main
Jul 9, 2024
Merged

Fix test_db_counters test#6447
jepett0 merged 2 commits intoydb-platform:mainfrom
jepett0:main

Conversation

@jepett0
Copy link
Contributor

@jepett0 jepett0 commented Jul 9, 2024

Ticket

Make the test independent of the FeatureFlags.EnableLocalDBBtreeIndex. The issue was discovered in the new stable-24-3 branch which had a different default value of the FeatureFlags.EnableLocalDBBtreeIndex.

- make it independent of the FeatureFlags.EnableLocalDBBtreeIndex
@jepett0 jepett0 requested a review from ijon July 9, 2024 06:39
@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@jepett0 jepett0 marked this pull request as ready for review July 9, 2024 06:47
@github-actions
Copy link

github-actions bot commented Jul 9, 2024

2024-07-09 06:57:24 UTC Pre-commit check for 54daf95 has started.
2024-07-09 06:59:23 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-07-09 07:25:06 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Jul 9, 2024

2024-07-09 06:57:35 UTC Pre-commit check for 54daf95 has started.
2024-07-09 06:59:35 UTC Build linux-x86_64-release-asan is running...
🟢 2024-07-09 07:16:19 UTC Build successful.
2024-07-09 07:16:34 UTC Tests are running...
🟢 2024-07-09 07:21:28 UTC Tests successful.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
77 74 0 0 2 1

🟢 2024-07-09 07:21:50 UTC ydbd size 5.2 GiB changed* by -557.0 KiB, which is <= 0 Bytes vs main: OK

ydbd size dash main: 8159d83 merge: 54daf95 diff diff %
ydbd size 5 611 510 480 Bytes 5 610 940 072 Bytes -557.0 KiB -0.010%
ydbd stripped size 1 206 639 512 Bytes 1 206 606 552 Bytes -32.2 KiB -0.003%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

@github-actions
Copy link

github-actions bot commented Jul 9, 2024

2024-07-09 07:02:42 UTC Pre-commit check for 54daf95 has started.
2024-07-09 07:04:53 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-07-09 07:18:25 UTC Build successful.
2024-07-09 07:18:39 UTC Tests are running...
🟢 2024-07-09 07:23:30 UTC Tests successful.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
78 75 0 0 2 1

🟢 2024-07-09 07:23:46 UTC ydbd size 8.1 GiB changed* by -1.1 MiB, which is <= 0 Bytes vs main: OK

ydbd size dash main: 8159d83 merge: 54daf95 diff diff %
ydbd size 8 721 689 568 Bytes 8 720 569 528 Bytes -1.1 MiB -0.013%
ydbd stripped size 475 008 736 Bytes 474 992 800 Bytes -15.6 KiB -0.003%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants