[GitHub] carbondata pull request #1919: [CARBONDATA-2123] Refactor datamap schema thr...

[xubo245]

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

    https://github.com/apache/carbondata/pull/1919#discussion_r165809708
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/datamap/CarbonCreateDataMapCommand.scala
 ---
    @@ -60,7 +61,14 @@ case class CarbonCreateDataMapCommand(
         } else {
           dmProperties
         }
    -
    +    val dataMapProvider = {
    +      try {
    +        DataMapProvider.getDataMapProvider(dmClassName)
    --- End diff --
    
    I understand user can provide className and shortName, both belong to 
provider name, right?


---