Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1595#discussion_r155458074
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
---
@@ -1168,7 +1168,7 @@ abstract class CarbonDDLSqlParser extends
AbstractCarbonSparkSQLParser {
* @param values
* @return
*/
- protected def parseDataType(dataType: String, values: Option[List[(Int,
Int)]]): DataTypeInfo = {
+ def parseDataType(dataType: String, values: Option[List[(Int, Int)]]):
DataTypeInfo = {
--- End diff --
seems wrong identation
---