-
Notifications
You must be signed in to change notification settings - Fork 940
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Port HIVE-27815: Support update numModifiedRows #6639
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #6639 +/- ##
======================================
Coverage 0.00% 0.00%
======================================
Files 680 680
Lines 42045 42056 +11
Branches 5739 5743 +4
======================================
- Misses 42045 42056 +11 ☔ View full report in Codecov by Sentry. |
kyuubi-hive-jdbc/src/main/java/org/apache/kyuubi/jdbc/hive/KyuubiStatement.java
Outdated
Show resolved
Hide resolved
interesting error, CI runs at mid-night at UTC timezone, thus failed with
let me re-trigger it. |
Merged to master |
Given this causes significant perf regression(see #6796) and the newly added functionality is not completed, let me revert it first. |
🔍 Description
Backport apache/hive#4819
Note: it's only the JDBC driver side change, to make it work, we also need to modify the engines.
Types of changes 🔖
Test Plan 🧪
We can not test this feature so far because we don't have engine support that. Pass GHA to ensure it breaks nothing.
Checklist 📝
Be nice. Be informative.