Skip to content

Prohibit UDF on Flow in case of PartitionByKeys callable. #6322

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.

Sign up for GitHub

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

Jump to bottom
Merged
Darych merged 1 commit into from
Jul 9, 2024
Merged

Prohibit UDF on Flow in case of PartitionByKeys callable. #6322

Darych merged 1 commit into from
Jul 9, 2024

Conversation

@Darych
Copy link
Contributor

@Darych Darych commented Jul 5, 2024

Changelog entry

Prohibit UDF on Flow in case of PartitionByKeys callable.

Changelog category

  • Bugfix

Additional information

...

@Darych Darych added area/yql YQL query language issues bugfix labels Jul 5, 2024
@Darych Darych requested review from rvu1024 and vitstn July 5, 2024 09:41
@Darych Darych self-assigned this Jul 5, 2024
@Darych Darych requested a review from a team as a code owner July 5, 2024 09:41
@github-actions
Copy link

github-actions bot commented Jul 5, 2024 •
edited
Loading

⚪ 2024-07-05 09:54:01 UTC Pre-commit check for cd83892 has started.
⚪ 2024-07-05 09:56:35 UTC Build linux-x86_64-release-clang14 is running...
🔴 2024-07-05 09:56:48 UTC Build failed. see the build logs.

@github-actions
Copy link

github-actions bot commented Jul 5, 2024 •
edited
Loading

⚪ 2024-07-05 10:03:10 UTC Pre-commit check for cd83892 has started.
⚪ 2024-07-05 10:06:05 UTC Build linux-x86_64-release-asan is running...
🔴 2024-07-05 10:06:19 UTC Build failed. see the build logs.
🔴 2024-07-05 10:06:20 UTC Tests run skipped.

@github-actions
Copy link

github-actions bot commented Jul 5, 2024 •
edited
Loading

⚪ 2024-07-05 10:07:54 UTC Pre-commit check for cd83892 has started.
⚪ 2024-07-05 10:11:09 UTC Build linux-x86_64-relwithdebinfo is running...
🔴 2024-07-05 10:11:24 UTC Build failed. see the build logs.
🔴 2024-07-05 10:11:25 UTC Tests run skipped.

@rvu1024 rvu1024 added the rebase-and-check Rebase PR with the current base branch and check label Jul 5, 2024
@github-actions github-actions bot removed the rebase-and-check Rebase PR with the current base branch and check label Jul 5, 2024
@github-actions
Copy link

github-actions bot commented Jul 5, 2024 •
edited
Loading

⚪ 2024-07-05 12:29:09 UTC Pre-commit check for f3c122c has started.
⚪ 2024-07-05 12:32:08 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-07-05 12:37:51 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Jul 5, 2024 •
edited
Loading

⚪ 2024-07-05 12:53:17 UTC Pre-commit check for f3c122c has started.
⚪ 2024-07-05 12:56:21 UTC Build linux-x86_64-release-asan is running...
🟢 2024-07-05 13:20:36 UTC Build successful.
⚪ 2024-07-05 13:20:46 UTC Tests are running...
🔴 2024-07-05 15:18:37 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
12322 11711 0 93 345 173

🟢 2024-07-05 15:19:20 UTC ydbd size 5.2 GiB changed* by +60.1 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 5fc09ca merge: f3c122c diff diff %
ydbd size 5 615 862 464 Bytes 5 615 924 016 Bytes +60.1 KiB +0.001%
ydbd stripped size 1 211 107 272 Bytes 1 211 119 240 Bytes +11.7 KiB +0.001%

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

@Darych Darych added the rebase-and-check Rebase PR with the current base branch and check label Jul 8, 2024
@github-actions github-actions bot removed the rebase-and-check Rebase PR with the current base branch and check label Jul 8, 2024
@github-actions
Copy link

github-actions bot commented Jul 8, 2024 •
edited
Loading

⚪ 2024-07-08 07:53:52 UTC Pre-commit check for 2b36844 has started.
⚪ 2024-07-08 07:56:25 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-07-08 08:02:22 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Jul 8, 2024 •
edited
Loading

⚪ 2024-07-08 07:53:56 UTC Pre-commit check for 2b36844 has started.
⚪ 2024-07-08 07:56:27 UTC Build linux-x86_64-release-asan is running...
🟢 2024-07-08 08:21:01 UTC Build successful.
⚪ 2024-07-08 08:21:17 UTC Tests are running...
🔴 2024-07-08 10:18:32 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
12329 11727 0 89 340 173

🟢 2024-07-08 10:19:23 UTC ydbd size 5.2 GiB changed* by +6.9 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 1f88b11 merge: 2b36844 diff diff %
ydbd size 5 614 135 680 Bytes 5 614 142 720 Bytes +6.9 KiB +0.000%
ydbd stripped size 1 209 991 064 Bytes 1 209 992 280 Bytes +1.2 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
Copy link

github-actions bot commented Jul 8, 2024 •
edited
Loading

⚪ 2024-07-08 07:54:04 UTC Pre-commit check for 2b36844 has started.
⚪ 2024-07-08 07:57:08 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-07-08 08:38:11 UTC Build successful.
⚪ 2024-07-08 08:38:24 UTC Tests are running...
🟢 2024-07-08 10:42:59 UTC Tests successful.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
73372 59931 0 0 13430 11

🟢 2024-07-08 10:43:39 UTC ydbd size 8.1 GiB changed* by +4.1 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 1f88b11 merge: 2b36844 diff diff %
ydbd size 8 721 801 400 Bytes 8 721 805 560 Bytes +4.1 KiB +0.000%
ydbd stripped size 476 624 376 Bytes 476 624 824 Bytes +448 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

@Darych Darych merged commit d4eee5c into ydb-platform:main Jul 9, 2024
@Darych Darych deleted the prohibit-flow-udf-partitions branch July 9, 2024 14:03
rvu1024 pushed a commit to rvu1024/ydb that referenced this pull request Jul 10, 2024
@Darych @rvu1024
Prohibit UDF on Flow in case of PartitionByKeys callable. (ydb-platfo…
3022493 
…rm#6322)
@rvu1024 rvu1024 mentioned this pull request Jul 10, 2024
Merged
This was referenced Jul 12, 2024
Merged
Merged
Merged
This was referenced Jul 22, 2024
Merged
Merged
Merged
This was referenced Jul 23, 2024
Merged
Merged
lll-phill-lll pushed a commit to lll-phill-lll/ydb that referenced this pull request Sep 6, 2024
@Darych @lll-phill-lll
Prohibit UDF on Flow in case of PartitionByKeys callable. (ydb-platfo…
50e11da 
…rm#6322)
lll-phill-lll pushed a commit to lll-phill-lll/ydb that referenced this pull request Sep 6, 2024
@Darych @lll-phill-lll
Prohibit UDF on Flow in case of PartitionByKeys callable. (ydb-platfo…
0fa60a6 
…rm#6322)
lll-phill-lll pushed a commit to lll-phill-lll/ydb that referenced this pull request Sep 6, 2024
@Darych @lll-phill-lll
Prohibit UDF on Flow in case of PartitionByKeys callable. (ydb-platfo…
0386d58 
…rm#6322)
lll-phill-lll pushed a commit to lll-phill-lll/ydb that referenced this pull request Sep 10, 2024
@Darych @lll-phill-lll
Prohibit UDF on Flow in case of PartitionByKeys callable. (ydb-platfo…
ad2b1b4 
…rm#6322)
lll-phill-lll pushed a commit to lll-phill-lll/ydb that referenced this pull request Sep 11, 2024
@Darych @lll-phill-lll
Prohibit UDF on Flow in case of PartitionByKeys callable. (ydb-platfo…
f305354 
…rm#6322)
lll-phill-lll pushed a commit that referenced this pull request Sep 12, 2024
@Darych @lll-phill-lll
Prohibit UDF on Flow in case of PartitionByKeys callable. (#6322)
a38d88d
lll-phill-lll pushed a commit to lll-phill-lll/ydb that referenced this pull request Sep 13, 2024
@Darych @lll-phill-lll
Prohibit UDF on Flow in case of PartitionByKeys callable. (ydb-platfo…
4eda93a 
…rm#6322)
lll-phill-lll pushed a commit to lll-phill-lll/ydb that referenced this pull request Sep 15, 2024
@Darych @lll-phill-lll
Prohibit UDF on Flow in case of PartitionByKeys callable. (ydb-platfo…
300be35 
…rm#6322)
lll-phill-lll pushed a commit to lll-phill-lll/ydb that referenced this pull request Sep 15, 2024
@Darych @lll-phill-lll
Prohibit UDF on Flow in case of PartitionByKeys callable. (ydb-platfo…
17579a7 
…rm#6322)
@lll-phill-lll lll-phill-lll mentioned this pull request Sep 18, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rvu1024 rvu1024 rvu1024 approved these changes

@vitstn vitstn Awaiting requested review from vitstn

Assignees

@Darych Darych

Labels

area/yql YQL query language issues bugfix

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Darych @rvu1024