Skip to content

Conversation

@Tony-Romanov
Copy link
Collaborator

No description provided.

@github-actions
Copy link

github-actions bot commented Dec 24, 2023

Note

This is an automated comment that will be appended during run.

🔴 linux-x86_64-release-asan: some tests FAILED for commit f6c6572.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
15619 15462 0 26 101 30

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit f6c6572.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
59402 50212 0 17 9137 36

@Tony-Romanov Tony-Romanov added the area/yql YQL query language issues label Dec 25, 2023
}
}

return KeepConstraints(ctx.ReplaceNode(node->Tail().TailPtr(), node->Tail().Head().Head(), std::move(sort)), *node, ctx);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Тут ничего не может быть кроме Chopped? Distinct/Uniq?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Distinct/Uniq должны выводится естественным образом из Chopped на входе.

@Tony-Romanov Tony-Romanov marked this pull request as ready for review December 25, 2023 11:10
@Tony-Romanov Tony-Romanov merged commit 6709bae into ydb-platform:main Dec 25, 2023
@Tony-Romanov Tony-Romanov deleted the constraint_fix branch December 25, 2023 11:16
@jepett0 jepett0 mentioned this pull request Dec 28, 2023
@StekPerepolnen StekPerepolnen mentioned this pull request Dec 29, 2023
Closed
adameat pushed a commit to adameat/ydb that referenced this pull request Dec 29, 2023
* Fix constrains after expand PartitionsByKeys.

* A little fix and cosmetic.
This was referenced Jan 3, 2024
This was referenced Jan 11, 2024
@vitstn vitstn mentioned this pull request Feb 16, 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants