[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-19 Thread ravipesala
Github user ravipesala commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
SDV Build Success , Please check CI 
http://144.76.159.231:8080/job/ApacheSDVTests/3008/



---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-19 Thread anubhav100
Github user anubhav100 commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
@jackylk  i accidently deleted the commits of this branch when squashing 
the commit i raised the new pull request with same changes please review #1839


---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-19 Thread jackylk
Github user jackylk commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
please squash and push again


---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-19 Thread jackylk
Github user jackylk commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
LGTM


---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-18 Thread anubhav100
Github user anubhav100 commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
@jackylk please review


---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-18 Thread ravipesala
Github user ravipesala commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
SDV Build Success , Please check CI 
http://144.76.159.231:8080/job/ApacheSDVTests/2970/



---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-18 Thread anubhav100
Github user anubhav100 commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
retest sdv please


---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-18 Thread ravipesala
Github user ravipesala commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
SDV Build Fail , Please check CI 
http://144.76.159.231:8080/job/ApacheSDVTests/2963/



---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-17 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
Build Success with Spark 2.1.0, Please check CI 
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/2921/



---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-17 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
Build Success with Spark 2.2.1, Please check CI 
http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/1688/



---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-17 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
Build Success with Spark 2.1.0, Please check CI 
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/2914/



---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-17 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
Build Failed with Spark 2.2.1, Please check CI 
http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/1681/



---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-17 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
Build Success with Spark 2.1.0, Please check CI 
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/2909/



---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-17 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
Build Failed with Spark 2.2.1, Please check CI 
http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/1676/



---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-17 Thread anubhav100
Github user anubhav100 commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
retest this please


---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-17 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
Build Failed  with Spark 2.1.0, Please check CI 
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/2904/



---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-17 Thread anubhav100
Github user anubhav100 commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
retest this please


---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-17 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
Build Failed with Spark 2.2.1, Please check CI 
http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/1672/



---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-17 Thread anubhav100
Github user anubhav100 commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
retest this please


---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-17 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
Build Failed  with Spark 2.1.0, Please check CI 
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/2896/



---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-17 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
Build Failed with Spark 2.2.1, Please check CI 
http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/1662/



---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-17 Thread anubhav100
Github user anubhav100 commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
@jackylk  

**here is the root reason**

**when we apply the alter table command with default value i;e
   "alter table customer1 add columns (intfield int) TBLPROPERTIES 
('DEFAULT.VALUE.intfield'='10')")

that value is always return as the long object in restructure util thats 
why compaction failed because the column value for newly added column is long 
but the column type is int**


it is wrongly written in restructure util  this code is wrong

  public static Object getMeasureDefaultValue(ColumnSchema columnSchema, 
byte[] defaultValue) {
Object measureDefaultValue = null;
if (!isDefaultValueNull(defaultValue)) {
  String value = null;
  DataType dataType = columnSchema.getDataType();
  if (dataType == DataTypes.SHORT || dataType == DataTypes.INT || 
dataType == DataTypes.LONG) {
value = new String(defaultValue, 
Charset.forName(CarbonCommonConstants.DEFAULT_CHARSET));
measureDefaultValue = Long.parseLong(value);
  } else if (DataTypes.isDecimal(dataType)) {
BigDecimal decimal = DataTypeUtil.byteToBigDecimal(defaultValue);
if (columnSchema.getScale() > decimal.scale()) {
  decimal = decimal.setScale(columnSchema.getScale(), 
RoundingMode.HALF_UP);
}
measureDefaultValue = decimal;
  } else {
value = new String(defaultValue, 
Charset.forName(CarbonCommonConstants.DEFAULT_CHARSET));
Double parsedValue = Double.valueOf(value);
if (!Double.isInfinite(parsedValue) && !Double.isNaN(parsedValue)) {
  measureDefaultValue = parsedValue;
}
  }
}
return measureDefaultValue;
  }

this code is always returning a long value objetc that is wrong

i corrected it as 

  */
  public static Object getMeasureDefaultValue(ColumnSchema columnSchema, 
byte[] defaultValue) {
Object measureDefaultValue = null;
if (!isDefaultValueNull(defaultValue)) {
  String value = null;
  DataType dataType = columnSchema.getDataType();
  if (dataType == DataTypes.SHORT) {
value = new String(defaultValue, 
Charset.forName(CarbonCommonConstants.DEFAULT_CHARSET));
measureDefaultValue = new Short(value);
  }
 else if (dataType == DataTypes.LONG) {
value = new String(defaultValue, 
Charset.forName(CarbonCommonConstants.DEFAULT_CHARSET));
measureDefaultValue = Long.parseLong(value);
  }
 else if (dataType == DataTypes.INT) {
value = new String(defaultValue, 
Charset.forName(CarbonCommonConstants.DEFAULT_CHARSET));
measureDefaultValue = Integer.parseInt(value);
  }
  else if (DataTypes.isDecimal(dataType)) {
BigDecimal decimal = DataTypeUtil.byteToBigDecimal(defaultValue);
if (columnSchema.getScale() > decimal.scale()) {
  decimal = decimal.setScale(columnSchema.getScale(), 
RoundingMode.HALF_UP);
}
measureDefaultValue = decimal;
  } else {
value = new String(defaultValue, 
Charset.forName(CarbonCommonConstants.DEFAULT_CHARSET));
Double parsedValue = Double.valueOf(value);
if (!Double.isInfinite(parsedValue) && !Double.isNaN(parsedValue)) {
  measureDefaultValue = parsedValue;
}
  }
}
return measureDefaultValue;
  }

please review i corrected it


---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-11 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
Build Success with Spark 2.1.0, Please check CI 
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/2712/



---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-11 Thread anubhav100
Github user anubhav100 commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
retest this please


---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-11 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
Build Failed with Spark 2.2.1, Please check CI 
http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/1477/



---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-11 Thread anubhav100
Github user anubhav100 commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
retest this please


---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-11 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
Build Failed  with Spark 2.1.0, Please check CI 
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/2709/



---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-11 Thread anubhav100
Github user anubhav100 commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
retest this please


---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-11 Thread ravipesala
Github user ravipesala commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
SDV Build Fail , Please check CI 
http://144.76.159.231:8080/job/ApacheSDVTests/2838/



---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-11 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
Build Success with Spark 2.1.0, Please check CI 
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/2706/



---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-11 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
Build Failed with Spark 2.2.1, Please check CI 
http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/1473/



---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-11 Thread anubhav100
Github user anubhav100 commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
retest this please


---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-11 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
Build Failed with Spark 2.2.1, Please check CI 
http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/1470/



---


[GitHub] carbondata issue #1795: [CARBONDATA-2016] Exception displays while executing...

2018-01-11 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue:

https://github.com/apache/carbondata/pull/1795
  
Build Success with Spark 2.1.0, Please check CI 
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/2703/



---