Skip to content

Join engine works with analyzer #43897

Merged
kitaisreal merged 1 commit intomasterfrom
vdimir/analyzer-storage-join
Dec 14, 2022
Merged

Join engine works with analyzer #43897
kitaisreal merged 1 commit intomasterfrom
vdimir/analyzer-storage-join

Conversation

@vdimir
Copy link
Copy Markdown
Member

@vdimir vdimir commented Dec 2, 2022

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

...

This PR also contains changes from #43963 and #44012 (because there are bugs in implementation not related to analyzer, so we need to fix them first)

@robot-ch-test-poll2 robot-ch-test-poll2 added the pr-not-for-changelog This PR should not be mentioned in the changelog label Dec 2, 2022
@vdimir vdimir force-pushed the vdimir/analyzer-storage-join branch from f230c3f to 23d7c42 Compare December 2, 2022 19:20
@vdimir vdimir marked this pull request as draft December 2, 2022 21:35
@vdimir vdimir force-pushed the vdimir/analyzer-storage-join branch 2 times, most recently from 206c361 to a64837b Compare December 9, 2022 17:24
@vdimir vdimir changed the title Fix column names produced by Join engine with analyzer Join engine works with analyzer Dec 9, 2022
@vdimir vdimir force-pushed the vdimir/analyzer-storage-join branch 2 times, most recently from 27f559a to d9c7094 Compare December 9, 2022 17:30
@vdimir vdimir force-pushed the vdimir/analyzer-storage-join branch from d9c7094 to a25c0d7 Compare December 12, 2022 12:26
@vdimir vdimir marked this pull request as ready for review December 13, 2022 12:08
@vdimir vdimir requested a review from kitaisreal December 13, 2022 12:08
@kitaisreal kitaisreal self-assigned this Dec 14, 2022
@kitaisreal kitaisreal merged commit 335f8ed into master Dec 14, 2022
@kitaisreal kitaisreal deleted the vdimir/analyzer-storage-join branch December 14, 2022 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants