[ 
https://issues.apache.org/jira/browse/CARBONDATA-267?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15535090#comment-15535090
 ] 

ASF GitHub Bot commented on CARBONDATA-267:
-------------------------------------------

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

    https://github.com/apache/incubator-carbondata/pull/189#discussion_r81280323
  
    --- Diff: 
integration/spark/src/main/scala/org/apache/spark/sql/CarbonSqlParser.scala ---
    @@ -546,6 +548,18 @@ class CarbonSqlParser()
         colGrpNames.toString()
       }
     
    +  protected def getTableBlockSize(tableProperties: Map[String, String]): 
Integer = {
    +    var tableBlockSize: Integer = 0
    +    if 
(tableProperties.get(CarbonCommonConstants.TABLE_BLOCKSIZE).isDefined) {
    +      val blockSizeStr: String = 
tableProperties.get(CarbonCommonConstants.TABLE_BLOCKSIZE).get
    +      try {
    +        tableBlockSize = Integer.parseInt(blockSizeStr)
    +      } catch {
    +        case e: NumberFormatException => tableBlockSize = 0
    --- End diff --
    
    better assign default value. so that system will be in a consistent state 
always


> Set block_size for table on table level
> ---------------------------------------
>
>                 Key: CARBONDATA-267
>                 URL: https://issues.apache.org/jira/browse/CARBONDATA-267
>             Project: CarbonData
>          Issue Type: New Feature
>    Affects Versions: 0.1.0-incubating
>            Reporter: zhangshunyu
>            Assignee: zhangshunyu
>             Fix For: 0.2.0-incubating
>
>
> Set block_size for table on table level



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to