[ https://issues.apache.org/jira/browse/CARBONDATA-306?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15574337#comment-15574337 ]
ASF GitHub Bot commented on CARBONDATA-306: ------------------------------------------- Github user jackylk commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/230#discussion_r83361557 --- Diff: integration/spark/src/main/scala/org/apache/spark/sql/execution/command/carbonTableSchema.scala --- @@ -1422,6 +1422,7 @@ private[sql] case class DescribeCommandFormatted( results ++= Seq(("Table Name : ", relation.tableMeta.carbonTableIdentifier.getTableName, "")) results ++= Seq(("CARBON Store Path : ", relation.tableMeta.storePath, "")) val carbonTable = relation.tableMeta.carbonTable + results ++= Seq(("Table Block Size : ", carbonTable.getBlocksize + " MB", "")) --- End diff -- If so, can you change the corresponding variable name and function name to indicate it is bytes in MB, like `getBlockSizeInMB` and add comment to `CarbonCommonConstants.TABLE_BLOCKSIZE` > block size info should be show in Desc Formatted and executor log > ----------------------------------------------------------------- > > Key: CARBONDATA-306 > URL: https://issues.apache.org/jira/browse/CARBONDATA-306 > Project: CarbonData > Issue Type: Improvement > Reporter: Jay > Priority: Minor > > when run desc formatted command, the table block size should be show, as well > as in executor log when run load command -- This message was sent by Atlassian JIRA (v6.3.4#6332)