search

apache / carbondata

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

optmize code smell in presto module #4331

Closed xubo245 closed 1 year ago

xubo245 commented 1 year ago

add override

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?

optmize code smell in presto module

Does this PR introduce any user interface change?