Skip to content

25-1: Preserve column order - #18938

Merged
CyberROFL merged 1 commit into
ydb-platform:stable-25-1from
CyberROFL:preserve-column-order-25-1
May 28, 2025
Merged

CyberROFL merged 1 commit into
ydb-platform:stable-25-1from
CyberROFL:preserve-column-order-25-1

Conversation

@CyberROFL

@CyberROFL CyberROFL commented May 28, 2025 •

Copy link
Copy Markdown
Member

Changelog entry

In the table description columns are returned in the same order as they were specified in CREATE TABLE.

Changelog category

  • Bugfix

Description for reviewers

Issue: #18622
Original pr: #18664

@github-actions

github-actions Bot commented May 28, 2025 •

Copy link
Copy Markdown
Contributor

⚪ 2025-05-28 11:08:04 UTC Pre-commit check linux-x86_64-release-asan for b7ce361 has started.
⚪ 2025-05-28 11:08:16 UTC Artifacts will be uploaded here
⚪ 2025-05-28 11:11:13 UTC ya make is running...
🟡 2025-05-28 12:27:37 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet Going to retry failed tests...

Details

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
12215 12081 0 34 63 37

⚪ 2025-05-28 12:28:39 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-05-28 12:49:15 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet Going to retry failed tests...

Details

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
170 (only retried tests) 73 0 7 54 36

⚪ 2025-05-28 12:49:23 UTC ya make is running... (failed tests rerun, try 3)
🟡 2025-05-28 13:09:21 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet

Test history | Ya make output | Test bloat | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
126 (only retried tests) 34 0 7 52 33

🟢 2025-05-28 13:09:28 UTC Build successful.
🟢 2025-05-28 13:09:54 UTC ydbd size 3.6 GiB changed* by -46.4 KiB, which is <= 0 Bytes vs stable-25-1: OK

ydbd size dash stable-25-1: 8b86cdb merge: b7ce361 diff diff %
ydbd size 3 912 905 384 Bytes 3 912 857 824 Bytes -46.4 KiB -0.001%
ydbd stripped size 1 368 555 792 Bytes 1 368 554 448 Bytes -1.3 KiB -0.000%

*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

github-actions Bot commented May 28, 2025 •

Copy link
Copy Markdown
Contributor

⚪ 2025-05-28 11:08:05 UTC Pre-commit check linux-x86_64-relwithdebinfo for b7ce361 has started.
⚪ 2025-05-28 11:08:16 UTC Artifacts will be uploaded here
⚪ 2025-05-28 11:11:11 UTC ya make is running...
🟡 2025-05-28 12:14:08 UTC Some tests failed, follow the links below. Going to retry failed tests...

Details

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
19262 17986 0 4 1207 65

⚪ 2025-05-28 12:15:45 UTC ya make is running... (failed tests rerun, try 2)
🟢 2025-05-28 12:35:39 UTC Tests successful.

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
130 (only retried tests) 70 0 0 2 58

🟢 2025-05-28 12:35:46 UTC Build successful.
🟢 2025-05-28 12:36:06 UTC ydbd size 2.1 GiB changed* by -7.3 KiB, which is <= 0 Bytes vs stable-25-1: OK

ydbd size dash stable-25-1: 8b86cdb merge: b7ce361 diff diff %
ydbd size 2 249 320 056 Bytes 2 249 312 608 Bytes -7.3 KiB -0.000%
ydbd stripped size 476 028 280 Bytes 476 027 960 Bytes -320 Bytes -0.000%

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

@CyberROFL
CyberROFL marked this pull request as ready for review May 28, 2025 11:08
@CyberROFL
CyberROFL requested a review from a team as a code owner May 28, 2025 11:08
@github-actions

github-actions Bot commented May 28, 2025 •

Copy link
Copy Markdown
Contributor

🟢 2025-05-28 11:13:18 UTC The validation of the Pull Request description is successful.

@CyberROFL
CyberROFL enabled auto-merge May 28, 2025 12:29
@CyberROFL CyberROFL self-assigned this May 28, 2025
@CyberROFL
CyberROFL merged commit 8571c01 into ydb-platform:stable-25-1 May 28, 2025
@CyberROFL
CyberROFL deleted the preserve-column-order-25-1 branch May 28, 2025 19:15
azevaykin added a commit to azevaykin/ydb that referenced this pull request May 29, 2025
azevaykin added a commit to azevaykin/ydb that referenced this pull request Jun 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Table with 29 columns has incorrect column order in DescribeTable result

3 participants