-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[StarRocks On ES] Support array format. #9693
Conversation
run starrocks_clang-format |
Signed-off-by: Smith Cruise <[email protected]>
array = down_cast<ArrayColumn*>(column); | ||
} | ||
|
||
auto* offsets = array->offsets_column().get(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should define offsets
when using it to understand easily.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
run starrocks_admit_test |
@Mergifyio backport branch-2.1 |
(cherry picked from commit 6ae4a62) # Conflicts: # be/test/CMakeLists.txt
✅ Backports have been created
|
@Mergifyio backport branch-2.3 |
(cherry picked from commit 6ae4a62)
✅ Backports have been created
|
@Mergifyio backport branch-2.4 |
(cherry picked from commit 6ae4a62)
✅ Backports have been created
|
(cherry picked from commit 6ae4a62)
(cherry picked from commit 6ae4a62)
Support array format in ES.
What type of PR is this:
Which issues of this PR fixes :
Fixes #
Problem Summary(Required) :
Checklist: