You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by ha...@apache.org on 2014/10/01 18:11:10 UTC
svn commit: r1628751 - in /hive/branches/branch-0.14:
metastore/src/java/org/apache/hadoop/hive/metastore/
ql/src/java/org/apache/hadoop/hive/ql/exec/
ql/src/test/results/clientpositive/
Author: hashutosh
Date: Wed Oct 1 16:11:10 2014
New Revision: 1628751
URL: http://svn.apache.org/r1628751
Log:
HIVE-8250 : Truncating table doesnt invalidate stats (Ashutosh Chauhan via Prashanth J)
Modified:
hive/branches/branch-0.14/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java
hive/branches/branch-0.14/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
hive/branches/branch-0.14/ql/src/test/results/clientpositive/alter_numbuckets_partitioned_table_h23.q.out
Modified: hive/branches/branch-0.14/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java
URL: http://svn.apache.org/viewvc/hive/branches/branch-0.14/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java?rev=1628751&r1=1628750&r2=1628751&view=diff
==============================================================================
--- hive/branches/branch-0.14/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java (original)
+++ hive/branches/branch-0.14/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java Wed Oct 1 16:11:10 2014
@@ -258,7 +258,7 @@ public class MetaStoreUtils {
if (oldPart.getParameters().containsKey(stat)) {
Long oldStat = Long.parseLong(oldPart.getParameters().get(stat));
Long newStat = Long.parseLong(newPart.getParameters().get(stat));
- if (oldStat != newStat) {
+ if (!oldStat.equals(newStat)) {
return true;
}
}
@@ -993,7 +993,7 @@ public class MetaStoreUtils {
partString = partString.concat(partStringSep);
partString = partString.concat(partKey.getName());
partTypesString = partTypesString.concat(partTypesStringSep);
- partTypesString = partTypesString.concat(partKey.getType());
+ partTypesString = partTypesString.concat(partKey.getType());
if (partStringSep.length() == 0) {
partStringSep = "/";
partTypesStringSep = ":";
@@ -1007,7 +1007,7 @@ public class MetaStoreUtils {
schema
.setProperty(
org.apache.hadoop.hive.metastore.api.hive_metastoreConstants.META_TABLE_PARTITION_COLUMN_TYPES,
- partTypesString);
+ partTypesString);
}
if (parameters != null) {
Modified: hive/branches/branch-0.14/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
URL: http://svn.apache.org/viewvc/hive/branches/branch-0.14/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java?rev=1628751&r1=1628750&r2=1628751&view=diff
==============================================================================
--- hive/branches/branch-0.14/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java (original)
+++ hive/branches/branch-0.14/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java Wed Oct 1 16:11:10 2014
@@ -4207,7 +4207,7 @@ public class DDLTask extends Task<DDLWor
String statVal = props.get(stat);
if (statVal != null && Long.parseLong(statVal) > 0) {
statsPresent = true;
- props.put(statVal, "0");
+ props.put(stat, "0");
props.put(StatsSetupConst.COLUMN_STATS_ACCURATE, "false");
}
}
Modified: hive/branches/branch-0.14/ql/src/test/results/clientpositive/alter_numbuckets_partitioned_table_h23.q.out
URL: http://svn.apache.org/viewvc/hive/branches/branch-0.14/ql/src/test/results/clientpositive/alter_numbuckets_partitioned_table_h23.q.out?rev=1628751&r1=1628750&r2=1628751&view=diff
==============================================================================
--- hive/branches/branch-0.14/ql/src/test/results/clientpositive/alter_numbuckets_partitioned_table_h23.q.out (original)
+++ hive/branches/branch-0.14/ql/src/test/results/clientpositive/alter_numbuckets_partitioned_table_h23.q.out Wed Oct 1 16:11:10 2014
@@ -358,11 +358,11 @@ Table: tst1
Protect Mode: None
#### A masked pattern was here ####
Partition Parameters:
- COLUMN_STATS_ACCURATE false
+ COLUMN_STATS_ACCURATE true
#### A masked pattern was here ####
numFiles 12
- numRows -1
- rawDataSize -1
+ numRows 500
+ rawDataSize 5312
totalSize 5812
#### A masked pattern was here ####