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

    https://github.com/apache/carbondata/pull/2404#discussion_r197816298
  
    --- Diff: 
integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/datamap/TestDataMapCommand.scala
 ---
    @@ -204,6 +204,37 @@ class TestDataMapCommand extends QueryTest with 
BeforeAndAfterAll {
         }
       }
     
    +  test("test show datamap: show datamap property related information") {
    +    val tableName = "datamapshowtest"
    +    val datamapName = "bloomdatamap"
    +    val datamapName2 = "bloomdatamap2"
    +    val datamapName3 = "bloomdatamap3"
    +    sql(s"drop table if exists $tableName")
    +    sql(s"create table $tableName (a string, b string, c string) stored by 
'carbondata'")
    --- End diff --
    
    no, we should not print the child select query to user, if DM properties is 
not given for preagg then you can show null, as we do in describe formatted for 
comment and all, which is the behavior of hive also.


---

Reply via email to