You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@carbondata.apache.org by manishgupta88 <gi...@git.apache.org> on 2017/12/15 07:16:30 UTC

[GitHub] carbondata pull request #1167: [CARBONDATA-1304] [IUD BuggFix] Iud with sing...

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

    https://github.com/apache/carbondata/pull/1167#discussion_r157134388
  
    --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/management/CarbonLoadDataCommand.scala ---
    @@ -314,14 +315,19 @@ case class CarbonLoadDataCommand(
         } else {
           None
         }
    +    var loadDataFrame = dataFrame
    --- End diff --
    
    Modify the code to make the variable as Val
    val loadDataFrame = if (updateModel.isDefined) {
           Some(getDataFrameWithTupleID)
         } else {
    		dataFrame
    	 }


---