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

Additional validation for queries with "json_list" format and datetime types #10208

Merged

Conversation

evanevanevanevannnn
Copy link
Collaborator

Changelog entry

Date, timestamp or time interval types are currently not supported with json_list format

Fix for YQ-3193

Changelog category

  • Bugfix

@evanevanevanevannnn evanevanevanevannnn requested a review from a team as a code owner October 8, 2024 12:08
@github-actions github-actions bot added the bugfix label Oct 8, 2024
Copy link

github-actions bot commented Oct 8, 2024

2024-10-08 12:12:38 UTC Pre-commit check linux-x86_64-relwithdebinfo for 7797a52 has started.
2024-10-08 12:12:49 UTC Artifacts will be uploaded here
2024-10-08 12:15:42 UTC ya make is running...
🟡 2024-10-08 13:17:25 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
74104 60158 0 2 13840 104

2024-10-08 13:24:44 UTC ya make is running... (failed tests rerun, try 2)
🟢 2024-10-08 13:36:19 UTC Tests successful.

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
104 (only retried tests) 4 0 0 0 100

🟢 2024-10-08 13:36:26 UTC Build successful.
🟢 2024-10-08 13:36:44 UTC ydbd size 2.8 GiB changed* by +41.0 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 8b39e36 merge: 7797a52 diff diff %
ydbd size 3 008 719 720 Bytes 3 008 761 728 Bytes +41.0 KiB +0.001%
ydbd stripped size 477 379 640 Bytes 477 388 600 Bytes +8.8 KiB +0.002%

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

Copy link

github-actions bot commented Oct 8, 2024

2024-10-08 12:12:44 UTC Pre-commit check linux-x86_64-release-asan for 7797a52 has started.
2024-10-08 12:12:55 UTC Artifacts will be uploaded here
2024-10-08 12:15:59 UTC ya make is running...
🟡 2024-10-08 13:52:01 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11333 10856 0 113 321 43

🟢 2024-10-08 13:53:02 UTC Build successful.
🟡 2024-10-08 13:53:34 UTC ydbd size 5.7 GiB changed* by +103.3 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: 8b39e36 merge: 7797a52 diff diff %
ydbd size 6 097 996 744 Bytes 6 098 102 496 Bytes +103.3 KiB +0.002%
ydbd stripped size 1 520 324 752 Bytes 1 520 352 848 Bytes +27.4 KiB +0.002%

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

Copy link

github-actions bot commented Oct 8, 2024

2024-10-08 15:03:33 UTC Pre-commit check linux-x86_64-relwithdebinfo for cce183e has started.
2024-10-08 15:04:07 UTC Artifacts will be uploaded here
2024-10-08 15:07:49 UTC ya make is running...
🟡 2024-10-08 16:08:51 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
74105 60155 0 3 13842 105

2024-10-08 16:15:39 UTC ya make is running... (failed tests rerun, try 2)
🟡 2024-10-08 16:27:21 UTC Some tests failed, follow the links below. Going to retry failed tests...

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
107 (only retried tests) 6 0 1 0 100

2024-10-08 16:27:29 UTC ya make is running... (failed tests rerun, try 3)
🔴 2024-10-08 16:39:03 UTC Some tests failed, follow the links below.

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
101 (only retried tests) 1 0 1 0 99

🟢 2024-10-08 16:39:09 UTC Build successful.
🟢 2024-10-08 16:39:34 UTC ydbd size 2.8 GiB changed* by +40.7 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 223be51 merge: cce183e diff diff %
ydbd size 3 008 718 152 Bytes 3 008 759 840 Bytes +40.7 KiB +0.001%
ydbd stripped size 477 379 768 Bytes 477 388 600 Bytes +8.6 KiB +0.002%

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

Copy link

github-actions bot commented Oct 8, 2024

2024-10-08 15:05:02 UTC Pre-commit check linux-x86_64-release-asan for cce183e has started.
2024-10-08 15:05:10 UTC Artifacts will be uploaded here
2024-10-08 15:08:03 UTC ya make is running...
🟡 2024-10-08 16:44:05 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11301 10804 0 121 327 49

🟢 2024-10-08 16:45:06 UTC Build successful.
🟡 2024-10-08 16:45:39 UTC ydbd size 5.7 GiB changed* by +102.2 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: 223be51 merge: cce183e diff diff %
ydbd size 6 097 995 312 Bytes 6 098 100 000 Bytes +102.2 KiB +0.002%
ydbd stripped size 1 520 325 008 Bytes 1 520 352 592 Bytes +26.9 KiB +0.002%

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

Copy link

github-actions bot commented Oct 9, 2024

2024-10-10 08:50:00 UTC Pre-commit check linux-x86_64-relwithdebinfo for e0b4c23 has started.
2024-10-10 08:50:12 UTC Artifacts will be uploaded here
2024-10-10 08:53:09 UTC ya make is running...
🔴 2024-10-10 09:00:33 UTC Build failed, see the logs.

Copy link

github-actions bot commented Oct 9, 2024

2024-10-09 16:00:58 UTC Pre-commit check linux-x86_64-release-asan for e0b4c23 has started.
2024-10-09 16:01:09 UTC Artifacts will be uploaded here
2024-10-09 16:04:06 UTC ya make is running...
🔴 2024-10-09 16:08:58 UTC Build failed, see the logs.

@evanevanevanevannnn evanevanevanevannnn added the rebase-and-check Rebase PR with the current base branch and check label Oct 10, 2024
@github-actions github-actions bot removed the rebase-and-check Rebase PR with the current base branch and check label Oct 10, 2024
Copy link

github-actions bot commented Oct 10, 2024

2024-10-10 10:13:25 UTC Pre-commit check linux-x86_64-relwithdebinfo for 23fdf72 has started.
2024-10-10 10:13:36 UTC Artifacts will be uploaded here
2024-10-10 10:16:35 UTC ya make is running...
🟡 2024-10-10 11:29:56 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
74128 60159 0 7 13858 104

2024-10-10 11:36:36 UTC ya make is running... (failed tests rerun, try 2)
🟢 2024-10-10 11:57:48 UTC Tests successful.

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
114 (only retried tests) 14 0 0 0 100

🟢 2024-10-10 11:57:55 UTC Build successful.
🟢 2024-10-10 11:58:17 UTC ydbd size 2.8 GiB changed* by +44.7 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: f6e0d96 merge: 23fdf72 diff diff %
ydbd size 3 012 083 536 Bytes 3 012 129 312 Bytes +44.7 KiB +0.002%
ydbd stripped size 477 604 856 Bytes 477 617 784 Bytes +12.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

Copy link

github-actions bot commented Oct 10, 2024

2024-10-10 10:14:08 UTC Pre-commit check linux-x86_64-release-asan for 23fdf72 has started.
2024-10-10 10:15:01 UTC Artifacts will be uploaded here
2024-10-10 10:18:30 UTC ya make is running...
🟡 2024-10-10 11:51:06 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11302 11080 0 59 58 105

🟢 2024-10-10 11:52:07 UTC Build successful.
🟡 2024-10-10 11:52:43 UTC ydbd size 5.7 GiB changed* by +102.2 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: f6e0d96 merge: 23fdf72 diff diff %
ydbd size 6 103 959 536 Bytes 6 104 064 208 Bytes +102.2 KiB +0.002%
ydbd stripped size 1 521 006 832 Bytes 1 521 034 416 Bytes +26.9 KiB +0.002%

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

@evanevanevanevannnn evanevanevanevannnn enabled auto-merge (squash) October 10, 2024 12:09
@evanevanevanevannnn evanevanevanevannnn merged commit a71cc84 into ydb-platform:main Oct 17, 2024
12 of 14 checks passed
@evanevanevanevannnn evanevanevanevannnn deleted the YQ_3193_fix_2 branch November 7, 2024 08:28
evanevanevanevannnn added a commit to evanevanevanevannnn/ydb that referenced this pull request Nov 7, 2024
evanevanevanevannnn added a commit to evanevanevanevannnn/ydb that referenced this pull request Nov 7, 2024
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.

4 participants