Skip to content
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

Disable Hive text native readers #18254

Closed

Conversation

kokosing
Copy link
Member

Disable Hive text native readers

There are some silent correctness issues related to Hive text
native readers. Once they got fixed we may want to turn them on again.

There are some silent correctness issues related to Hive text
native readers. Once they got fixed we may want to turn them on again.
@findepi
Copy link
Member

findepi commented Jul 12, 2023

Do we know of any correctness bugs that we cannot just fix right away?

@kokosing
Copy link
Member Author

I only know about #18215 and #18255. So disabling could be premature. Depends on the confidence we have in native readers. I posted this PR just to facilitate discussion.

@findinpath
Copy link
Contributor

@kokosing #18223 this one is also a newly discovered issue on this topic.

@github-actions github-actions bot added tests:hive hive Hive connector labels Jul 12, 2023
@kokosing
Copy link
Member Author

All mentioned things are closed, so let's close this one too.

@kokosing kokosing closed this Jul 14, 2023
@kokosing kokosing deleted the origin/master/153_disable-readers branch July 14, 2023 06:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed hive Hive connector
Development

Successfully merging this pull request may close these issues.

3 participants