QiangCai opened a new pull request #3782:
URL: https://github.com/apache/carbondata/pull/3782
### Why is this PR needed?
1. pointless bitwise expression
2. field can be local
3. standard Charset object can be used
4. unecessary conversion to String
5. unecessary interface modifier
6. unecessary semicolon
7. duplicate condition in 'if' statement
8. 'if' statement with common parts
9. Redundant 'if' statement
10. unnecessary 'null' check before method call
11. Redundant local variable
12. unused import
### What changes were proposed in this PR?
### Does this PR introduce any user interface change?
- No
- Yes. (please explain the change and update document)
### Is any new testcase added?
- No
- Yes
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
