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

Enable NarrowCalculation error-prone check #18479

Conversation

findepi
Copy link
Member

@findepi findepi commented Aug 1, 2023

This mostly matches IntelliJ's integer multiplication implicitly cast to long inspection. While places changed do not look like bugs, so can be viewed as false positives, this helps avoid IDE warnings and thus makes spotting problems easier.

findepi added 2 commits August 1, 2023 12:10
The floor argument is an int.
This mostly matches IntelliJ's `integer multiplication implicitly cast
to long` inspection. While places changed do not look like bugs, so can
be viewed as false positives, this helps avoid IDE warnings and thus
makes spotting problems easier.
@findepi findepi added the no-release-notes This pull request does not require release notes entry label Aug 1, 2023
@findepi findepi force-pushed the findepi/enable-narrowcalculation-error-prone-check-ff0d37 branch from 679816f to 34bf502 Compare August 1, 2023 10:13
@cla-bot cla-bot bot added the cla-signed label Aug 1, 2023
@github-actions github-actions bot added jdbc Relates to Trino JDBC driver tests:hive hive Hive connector labels Aug 1, 2023
@findepi findepi merged commit a327d63 into trinodb:master Aug 8, 2023
@findepi findepi deleted the findepi/enable-narrowcalculation-error-prone-check-ff0d37 branch August 8, 2023 10:55
@github-actions github-actions bot added this to the 423 milestone Aug 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed hive Hive connector jdbc Relates to Trino JDBC driver no-release-notes This pull request does not require release notes entry
Development

Successfully merging this pull request may close these issues.

3 participants