You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by "Pankit Thapar (JIRA)" <ji...@apache.org> on 2014/11/24 22:40:13 UTC
[jira] [Created] (HIVE-8955) alter partition should check for
"hive.stats.autogather" in hiveConf
Pankit Thapar created HIVE-8955:
-----------------------------------
Summary: alter partition should check for "hive.stats.autogather" in hiveConf
Key: HIVE-8955
URL: https://issues.apache.org/jira/browse/HIVE-8955
Project: Hive
Issue Type: Improvement
Components: Metastore
Affects Versions: 0.13.1
Reporter: Pankit Thapar
Fix For: 0.15.0
When alter partition code path is triggered, it should check for the flag "hive.stats.autogather", if it is true, then only updateStats else skip them.
This is done in append_partition code flow.
Is there any specific reason the alter_partition does not respect this conf variable?
//code snippet : HiveMetastore.java
private Partition append_partition_common(RawStore ms, String dbName, String tableName,
List<String> part_vals, EnvironmentContext envContext) throws InvalidObjectException,
AlreadyExistsException, MetaException {
...
....
if (HiveConf.getBoolVar(hiveConf, HiveConf.ConfVars.HIVESTATSAUTOGATHER) &&
!MetaStoreUtils.isView(tbl)) {
MetaStoreUtils.updatePartitionStatsFast(part, wh, madeDir);
}
...
...
}
The above code snippet checks for the variable but this same check is absent in
//code snippet : HiveAlterHandler.java
public Partition alterPartition(final RawStore msdb, Warehouse wh, final String dbname,
final String name, final List<String> part_vals, final Partition new_part)
throws InvalidOperationException, InvalidObjectException, AlreadyExistsException,
MetaException {
....
...
}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)