-
Notifications
You must be signed in to change notification settings - Fork 761
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
Feature: implement HiveSource to integrate with new processor #6588
Comments
We plan to remove the old processor codes #6584, FYI @sandflee |
plan to support the new processor in last two weeks. cc @dantengsky |
new processor HiveSource is implemented in #6584 |
@zhang2014 thanks for your work, do you mind to move HiveSource related code to a new PR, we could test and discuss in new PR, #6584 is too large. |
Agreed. And I think we can apply projection pushdown into |
This is difficult, but they are all in one commit(f9c68cd) and welcome add some comment in this commit. |
I think we can wait for @zhang2014 to finish his migration and then start the next work. |
It's ready for review. |
Summary
Description for this feature.
After #5895
HiveSource
is a dummy implementation, so we must turn offenable_new_processor_framework
to query hive data.It's better to make
HiveTable
read support the new processor.The text was updated successfully, but these errors were encountered: