-
Notifications
You must be signed in to change notification settings - Fork 703
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[CARBONDATA-3227] Fix some spell errors in the project #3052
Conversation
Can one of the admins verify this patch? |
add to whitelist |
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
Outdated
Show resolved
Hide resolved
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
Outdated
Show resolved
Hide resolved
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
Outdated
Show resolved
Hide resolved
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
Outdated
Show resolved
Hide resolved
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
Outdated
Show resolved
Hide resolved
add to whitelist |
Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/2177/ |
@XiaotaoYi Please optimize the title, like: [CARBONDATA-3227] Fix some spell errors in the project |
Build Failed with Spark 2.3.2, Please check CI http://136.243.101.176:8080/job/carbondataprbuilder2.3/10433/ |
Build Failed with Spark 2.2.1, Please check CI http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/2392/ |
@XiaotaoYi Please fix the CI errors. |
Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/2203/ |
Build Success with Spark 2.2.1, Please check CI http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/2420/ |
Build Success with Spark 2.3.2, Please check CI http://136.243.101.176:8080/job/carbondataprbuilder2.3/10460/ |
LGTM! Thanks for you contribution! |
LGTM |
Correct scala variable name in three files: carbondata\integration\spark-common\src\main\scala\org\apache\spark\sql\catalyst\CarbonDDLSqlParser.scala carbondata\integration\spark-common\src\main\scala\org\apache\carbondata\spark\util\CarbonScalaUtil.scala carbondata\integration\spark2\src\main\scala\org\apache\spark\util\AlterTableUtil.scala Change variable name from escapechar -> escapeChar optionlist -> optionList hivedefaultpartition -> hiveDefaultPartition pvalue -> pValue errormsg -> errorMsg isDetectAsDimentionDatatype -> isDetectAsDimentionDataType This closes #3052
Correct scala variable name in three files: carbondata\integration\spark-common\src\main\scala\org\apache\spark\sql\catalyst\CarbonDDLSqlParser.scala carbondata\integration\spark-common\src\main\scala\org\apache\carbondata\spark\util\CarbonScalaUtil.scala carbondata\integration\spark2\src\main\scala\org\apache\spark\util\AlterTableUtil.scala Change variable name from escapechar -> escapeChar optionlist -> optionList hivedefaultpartition -> hiveDefaultPartition pvalue -> pValue errormsg -> errorMsg isDetectAsDimentionDatatype -> isDetectAsDimentionDataType This closes apache#3052
Correct scala variable name in three files:
carbondata\integration\spark-common\src\main\scala\org\apache\spark\sql\catalyst\CarbonDDLSqlParser.scala
carbondata\integration\spark-common\src\main\scala\org\apache\carbondata\spark\util\CarbonScalaUtil.scala
carbondata\integration\spark2\src\main\scala\org\apache\spark\util\AlterTableUtil.scala
Change variable name from
escapechar -> escapeChar
optionlist -> optionList
hivedefaultpartition -> hiveDefaultPartition
pvalue -> pValue
errormsg -> errorMsg
isDetectAsDimentionDatatype -> isDetectAsDimentionDataType
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:
No
No
No
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.
No
No