hive icon indicating copy to clipboard operation
hive copied to clipboard

HIVE-28135: Remove HiveIOExceptionHandlerUtil

Open abstractdog opened this issue 1 year ago • 4 comments

What changes were proposed in this pull request?

Remove HiveIOExceptionHandlerUtil and all of its usage.

Why are the changes needed?

Because that's confusing and there is no clear use case, also other issues like HIVE-28133

Does this PR introduce any user-facing change?

If they hacked with "hive.io.exception.handlers", it will simply stop working.

Is the change a dependency upgrade?

No.

How was this patch tested?

Not used, not tested, precommit coverage should be fine.

abstractdog avatar Mar 26 '24 07:03 abstractdog

Quality Gate Passed Quality Gate passed

Issues
4 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

sonarqubecloud[bot] avatar Mar 29 '24 20:03 sonarqubecloud[bot]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Feel free to reach out on the [email protected] list if the patch is in need of reviews.

github-actions[bot] avatar May 29 '24 00:05 github-actions[bot]

rebasing the patch before merge

abstractdog avatar May 30 '24 09:05 abstractdog

Quality Gate Passed Quality Gate passed

Issues
4 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

sonarqubecloud[bot] avatar May 30 '24 10:05 sonarqubecloud[bot]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Feel free to reach out on the [email protected] list if the patch is in need of reviews.

github-actions[bot] avatar Jul 30 '24 00:07 github-actions[bot]

hm, we forgot about this, I'm reopening

abstractdog avatar Sep 04 '25 06:09 abstractdog

PR is green, this is ready for review

abstractdog avatar Sep 10 '25 09:09 abstractdog