### _Why are the changes needed?_ - remove duplicated assignment for the same variable in adjacent lines in `FastHiveDecimalImpl` - replace redundant `putAll` with collection initialization in `BatchRestApi` - use `try-with-resources` statement with the reader and avoid declaring two variables in the same line of code in `KyuubiCommands` - fix `warning: Tag 'return:' is not recognised` compilation warning in `KyuubiGetSqlClassification:L53` ### _How was this patch tested?_ - [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible - [ ] Add screenshots for manual tests if appropriate - [x] [Run test](https://kyuubi.readthedocs.io/en/master/contributing/code/testing.html#running-tests) locally before make a pull request Closes #5117 from bowenliang123/fastsignum. Closes #5117 595b5747d [liangbowen] simplify be530fac4 [liangbowen] fix warning: Tag '@return:' is not recognised compilation warning in KyuubiGetSqlClassification:L53 249706905 [liangbowen] use try-with-resources in KyuubiCommands a54a97fdd [liangbowen] remove redundant addAll call to collection initialization cc76d5d0f [liangbowen] remove repeated assignment Authored-by: liangbowen <liangbowen@gf.com.cn> Signed-off-by: Cheng Pan <chengpan@apache.org> |
||
|---|---|---|
| .. | ||
| src | ||
| pom.xml | ||