Github user jackylk commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2476#discussion_r202544380
  
    --- Diff: 
datamap/mv/core/src/main/scala/org/apache/carbondata/mv/datamap/MVHelper.scala 
---
    @@ -119,7 +119,12 @@ object MVHelper {
       }
     
       def updateColumnName(attr: Attribute): String = {
    -    val name = attr.name.replace("(", "_").replace(")", "").replace(" ", 
"_").replace("=", "")
    +    val name =
    +      attr.name.replace("(", "_").
    --- End diff --
    
    please put `.` to next line


---

Reply via email to