[GitHub] [carbondata] qiuchenjian commented on a change in pull request #3156: [CARBONDATA-3320]fix number of partitions issue in describe formatted

[GitBox <gi...@apache.org>]

qiuchenjian commented on a change in pull request #3156: [CARBONDATA-3320]fix number of partitions issue in describe formatted
URL: https://github.com/apache/carbondata/pull/3156#discussion_r267632939
 
 

 ##########
 File path: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/table/CarbonDescribeFormattedCommand.scala
 ##########
 @@ -239,6 +240,23 @@ private[sql] case class CarbonDescribeFormattedCommand(
     results.map{case (c1, c2, c3) => Row(c1, c2, c3)}
   }
 
+  /**
+   * This method returns the number of partitions based on the partition type
+   */
+  private def getNumberOfPartitions(carbonTable: CarbonTable,
+      sparkSession: SparkSession): String = {
+    val numberOfPartitions = carbonTable.getPartitionInfo.getNumPartitions
 
 Review comment:
   move to "case _=>" branch

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services