search

apache / carbondata

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

[ISSUE-4305] Optimize the constants and variable style #4311

Closed xubo245 closed 1 year ago

xubo245 commented 1 year ago

Why is this PR needed?

CONSTANTS should be like: UPPER_NAME; variable should be lowerCamelCase

What changes were proposed in this PR?

CONSTANTS should be like: UPPER_NAME;

variable should be lowerCamelCase

Does this PR introduce any user interface change?

shenjiayu17 commented 1 year ago

Build Failed with Spark 2.3.4, Please check CI http://159.138.8.58:12602/job/ApacheCarbonPRBuilder2.3/6380/

shenjiayu17 commented 1 year ago

Build Failed with Spark 2.4.5, Please check CI http://159.138.8.58:12602/job/ApacheCarbon_PR_Builder_2.4.5/4672/

shenjiayu17 commented 1 year ago

Build Failed with Spark 3.1, Please check CI http://159.138.8.58:12602/job/ApacheCarbon_PR_Builder_3.1/764/

chenliang613 commented 1 year ago

LGTM