-
Notifications
You must be signed in to change notification settings - Fork 122
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
Implements Hive Input/Output format to use vineyard as the storage backend #1520
Conversation
✅ Deploy Preview for v6d ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
/cc @vegetableysm
A quick note about string array and large string array: string array cannot contains chars more than. 2^32-1 while large string array can.
In vineyard, vineyard::StringArray
is arrow's large string array (LargeVarCharArray in java) and you shouldn't put string array to vineyard in any time. When hive gives you string array, cast it to large string array first then put to vineyard.
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.
@vegetableysm please
Thanks! |
performance test prepare data
insert test
select test
|
6e99f0b
to
69fa744
Compare
…ckend Signed-off-by: vegetableysm <yuanshumin.ysm@alibaba-inc.com> Signed-off-by: Tao He <linzhu.ht@alibaba-inc.com>
…ters Signed-off-by: Tao He <linzhu.ht@alibaba-inc.com>
Signed-off-by: Tao He <linzhu.ht@alibaba-inc.com>
5aac944
to
72bd707
Compare
What do these changes do?
Related issue number
Fixes #1420
Refer to:
PR #1554
PR #1551
PR #1548
PR #1552