hive icon indicating copy to clipboard operation
hive copied to clipboard

HIVE-27512: CalciteSemanticException.UnsupportedFeature enum to capital

Open maheshrajus opened this issue 2 years ago • 1 comments

What changes were proposed in this pull request?

Modified enum fields to capital letters and removed un used enums.

Why are the changes needed?

For better code readability and maintainability.

Does this PR introduce any user-facing change?

No

Is the change a dependency upgrade?

No

How was this patch tested?

Exists CI runs

maheshrajus avatar Nov 22 '23 13:11 maheshrajus

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 6 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

warning The version of Java (11.0.8) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17. Read more here

sonarqubecloud[bot] avatar Nov 23 '23 07:11 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 Mar 18 '24 00:03 github-actions[bot]

Quality Gate Passed Quality Gate passed

Issues
6 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 Apr 03 '24 05:04 sonarqubecloud[bot]

@ayushtkn thanks for the review. It is not any functional change. i just put the enum in proper readable format. If any other module is using this ENUM directly then better we can not disturb it. it causes compilation issues for other modules which are importing this enums directly.

maheshrajus avatar Apr 03 '24 08:04 maheshrajus

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 Jun 03 '24 00:06 github-actions[bot]