apache / carbondata

High performance data store solution
carbondata.apache.org
Apache License 2.0
1.43k stars 704 forks source link

[ISSUE-4329] optimize some code smells in presto module #4330

Closed xubo245 closed 9 months ago

xubo245 commented 11 months ago

static method shouldn't be called by object, it should be call by class

Why is this PR needed?

There are some code smells in presto module For example 1.Didn't add @OverRide for override method 2.magic number 3.variable Should Use LowerCamelCase

What changes were proposed in this PR?

optimize some code smells in presto module

Does this PR introduce any user interface change?

chenliang613 commented 9 months ago

LGTM