[
https://issues.apache.org/jira/browse/HIVE-21818?focusedWorklogId=255427&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-255427
]
ASF GitHub Bot logged work on HIVE-21818:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 06/Jun/19 20:49
Start Date: 06/Jun/19 20:49
Worklog Time Spent: 10m
Work Description: vineetgarg02 commented on pull request #663: HIVE-21818
URL: https://github.com/apache/hive/pull/663#discussion_r291362481
##########
File path: ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java
##########
@@ -5172,6 +5171,27 @@ private QBParseInfo getQBParseInfo(QB qb) throws
CalciteSemanticException {
}
}
+ @Override
+ protected Table getTableObjectByName(String tableName, boolean
throwException) throws HiveException {
+ if (!tabNameToTabObject.containsKey(tableName)) {
Review comment:
What happens if although tableName is same but they are different tables
belonging to different database?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Issue Time Tracking
-------------------
Worklog Id: (was: 255427)
Time Spent: 0.5h (was: 20m)
> CBO: Copying TableRelOptHiveTable has metastore traffic
> -------------------------------------------------------
>
> Key: HIVE-21818
> URL: https://issues.apache.org/jira/browse/HIVE-21818
> Project: Hive
> Issue Type: Bug
> Components: CBO
> Reporter: Gopal V
> Assignee: Jesus Camacho Rodriguez
> Priority: Major
> Labels: pull-request-available
> Attachments: HIVE-21818.01.patch, HIVE-21818.02.patch,
> HIVE-21818.03.patch, HIVE-21818.04.patch, HIVE-21818.04.patch,
> HIVE-21818.05.patch, HIVE-21818.05.patch, HIVE-21818.05.patch,
> HIVE-21818.patch
>
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
> While the optimizer is running, whenever it makes a copy of the TableScan to
> perform potential rewrites, there is Hive metastore traffic from inside CBO
> optimizer.
> {code}
> public RelOptHiveTable(RelOptSchema calciteSchema, RelDataTypeFactory
> typeFactory, List<String> qualifiedTblName,
> RelDataType rowType, Table hiveTblMetadata, List<ColumnInfo>
> hiveNonPartitionCols,
> List<ColumnInfo> hivePartitionCols, List<VirtualColumn>
> hiveVirtualCols, HiveConf hconf,
> Map<String, PrunedPartitionList> partitionCache, Map<String,
> ColumnStatsList> colStatsCache,
> AtomicInteger noColsMissingStats) { ....
> Pair<List<ImmutableBitSet>, List<ImmutableBitSet>> constraintKeys =
> generateKeys();
> this.keys = constraintKeys.left;
> this.nonNullablekeys = constraintKeys.right;
> this.referentialConstraints = generateReferentialConstraints();
> }
> {code}
> This is triggered from the rules as the partition pruner
> {code}
> protected void perform(RelOptRuleCall call, Filter filter,
> HiveTableScan tScan) {
> // Original table
> RelOptHiveTable hiveTable = (RelOptHiveTable) tScan.getTable();
> // Copy original table scan and table
> HiveTableScan tScanCopy = tScan.copyIncludingTable(tScan.getRowType());
> RelOptHiveTable hiveTableCopy = (RelOptHiveTable) tScanCopy.getTable();
> ...
> if (StringUtils.equals(hiveTableCopy.getPartitionListKey(),
> hiveTable.getPartitionListKey())) {
> // Nothing changed, we do not need to produce a new expression
> return;
> }
> {code}
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
