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

[dq] Validate self join with map strategy #4613

Merged
merged 2 commits into from
May 17, 2024

Conversation

rvu1024
Copy link
Member

@rvu1024 rvu1024 commented May 16, 2024

Changelog entry

Rejecting DQ execution of graphs with self join, DqReplicate without spilling, and disabled grace join. Searching rhombs in the graph if DqPhyMapJoin is present.

Changelog category

  • Not for changelog (changelog entry is not required)

Additional information

...

@rvu1024 rvu1024 added the area/yql YQL query language issues label May 16, 2024
@rvu1024 rvu1024 marked this pull request as ready for review May 16, 2024 13:44
@rvu1024 rvu1024 requested a review from a team as a code owner May 16, 2024 13:44
Copy link

github-actions bot commented May 16, 2024

2024-05-16 13:57:34 UTC Pre-commit check for 3f35723 has started.
2024-05-16 13:57:36 UTC Build linux-x86_64-release-clang14 is running...
🔴 2024-05-16 14:01:02 UTC Build failed. see the build logs.

Copy link

github-actions bot commented May 16, 2024

2024-05-16 14:01:27 UTC Pre-commit check for 3f35723 has started.
2024-05-16 14:01:29 UTC Build linux-x86_64-release-asan is running...
🔴 2024-05-16 14:05:44 UTC Build failed. see the build logs.
🔴 2024-05-16 14:07:33 UTC Tests run skipped.

Copy link

github-actions bot commented May 16, 2024

2024-05-16 14:29:59 UTC Pre-commit check for e3a8017 has started.
2024-05-16 14:30:01 UTC Build linux-x86_64-release-asan is running...
🟢 2024-05-16 14:32:46 UTC Build successful.
2024-05-16 14:34:26 UTC Tests are running...
🔴 2024-05-16 16:32:12 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
10122 9924 0 62 109 27

Copy link

github-actions bot commented May 16, 2024

2024-05-16 14:34:15 UTC Pre-commit check for e3a8017 has started.
2024-05-16 14:34:16 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-05-16 14:38:42 UTC Build successful.
2024-05-16 14:40:20 UTC Tests are running...
🔴 2024-05-16 16:32:12 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
68943 56213 0 14 12705 11

Copy link

github-actions bot commented May 16, 2024

2024-05-16 14:36:49 UTC Pre-commit check for e3a8017 has started.
2024-05-16 14:36:51 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-05-16 14:40:13 UTC Build successful.

@rvu1024 rvu1024 enabled auto-merge (squash) May 17, 2024 12:24
@rvu1024 rvu1024 merged commit ac286c8 into ydb-platform:main May 17, 2024
3 of 5 checks passed
@rvu1024 rvu1024 deleted the dq_validate_self_join branch May 17, 2024 15:06
MrLolthe1st pushed a commit to MrLolthe1st/ydb that referenced this pull request May 28, 2024
@niksaveliev niksaveliev mentioned this pull request May 29, 2024
This was referenced Jun 7, 2024
@niksaveliev niksaveliev mentioned this pull request Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/yql YQL query language issues not-for-changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants