[hive] branch master updated: HIVE-22816 : QueryCache: Queries using views can have them cached after CTE expansion (Gopal V via Jesus Camacho Rodriguez)

2020-02-20 Thread hashutosh
This is an automated email from the ASF dual-hosted git repository.

hashutosh pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git


The following commit(s) were added to refs/heads/master by this push:
 new f826283  HIVE-22816 : QueryCache: Queries using views can have them 
cached after CTE expansion (Gopal V via Jesus Camacho Rodriguez)
f826283 is described below

commit f826283d36503eb7598b1e33b10bd5d7d1ddd074
Author: Gopal V 
AuthorDate: Thu Feb 20 20:01:01 2020 -0800

HIVE-22816 : QueryCache: Queries using views can have them cached after CTE 
expansion (Gopal V via Jesus Camacho Rodriguez)

Signed-off-by: Ashutosh Chauhan 
---
 ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 
b/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
index f46739e..2328eed 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
@@ -15279,8 +15279,10 @@ public class SemanticAnalyzer extends 
BaseSemanticAnalyzer {
   }
 
   private List getNonTransactionalTables() {
+// views have been expanded by CBO already and can be ignored
 return tablesFromReadEntities(inputs)
 .stream()
+.filter(table -> !table.isView())
 .filter(table -> !AcidUtils.isTransactionalTable(table))
 .collect(Collectors.toList());
   }



[hive] branch master updated: HIVE-22880 : ACID: All delete event readers should ignore ORC SARGs (Gopal V via Rajesh Balamohan)

2020-02-20 Thread hashutosh
This is an automated email from the ASF dual-hosted git repository.

hashutosh pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git


The following commit(s) were added to refs/heads/master by this push:
 new 657b510  HIVE-22880 : ACID: All delete event readers should ignore ORC 
SARGs (Gopal V via Rajesh Balamohan)
657b510 is described below

commit 657b5107a34a205aae76fde42020453f04f40ead
Author: Gopal V 
AuthorDate: Thu Feb 20 19:26:51 2020 -0800

HIVE-22880 : ACID: All delete event readers should ignore ORC SARGs (Gopal 
V via Rajesh Balamohan)

Signed-off-by: Ashutosh Chauhan 
---
 .../apache/hadoop/hive/ql/io/orc/VectorizedOrcAcidRowBatchReader.java| 1 +
 1 file changed, 1 insertion(+)

diff --git 
a/ql/src/java/org/apache/hadoop/hive/ql/io/orc/VectorizedOrcAcidRowBatchReader.java
 
b/ql/src/java/org/apache/hadoop/hive/ql/io/orc/VectorizedOrcAcidRowBatchReader.java
index 2543dc6..0361872 100644
--- 
a/ql/src/java/org/apache/hadoop/hive/ql/io/orc/VectorizedOrcAcidRowBatchReader.java
+++ 
b/ql/src/java/org/apache/hadoop/hive/ql/io/orc/VectorizedOrcAcidRowBatchReader.java
@@ -236,6 +236,7 @@ public class VectorizedOrcAcidRowBatchReader
 // Set the range on the deleteEventReaderOptions to 0 to INTEGER_MAX 
because
 // we always want to read all the delete delta files.
 deleteEventReaderOptions.range(0, Long.MAX_VALUE);
+deleteEventReaderOptions.searchArgument(null, null);
 keyInterval = findMinMaxKeys(orcSplit, conf, deleteEventReaderOptions);
 DeleteEventRegistry der;
 try {



[hive] branch master updated: HIVE-22908: AM caching connections to LLAP based on hostname and port does not work in kubernetes (Prasanth Jayachandran reviewed by Gopal Vijayaraghavan)

2020-02-20 Thread prasanthj
This is an automated email from the ASF dual-hosted git repository.

prasanthj pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git


The following commit(s) were added to refs/heads/master by this push:
 new 703cf29  HIVE-22908: AM caching connections to LLAP based on hostname 
and port does not work in kubernetes (Prasanth Jayachandran reviewed by Gopal 
Vijayaraghavan)
703cf29 is described below

commit 703cf29fa14fffe7a7e0ee30cd11d16a5245bf1c
Author: Prasanth Jayachandran 
AuthorDate: Thu Feb 20 14:38:33 2020 -0800

HIVE-22908: AM caching connections to LLAP based on hostname and port does 
not work in kubernetes (Prasanth Jayachandran reviewed by Gopal Vijayaraghavan)
---
 .../org/apache/hadoop/hive/llap/AsyncPbRpcProxy.java | 16 +++-
 1 file changed, 15 insertions(+), 1 deletion(-)

diff --git 
a/llap-common/src/java/org/apache/hadoop/hive/llap/AsyncPbRpcProxy.java 
b/llap-common/src/java/org/apache/hadoop/hive/llap/AsyncPbRpcProxy.java
index ad39963..25e10f4 100644
--- a/llap-common/src/java/org/apache/hadoop/hive/llap/AsyncPbRpcProxy.java
+++ b/llap-common/src/java/org/apache/hadoop/hive/llap/AsyncPbRpcProxy.java
@@ -15,6 +15,8 @@
 package org.apache.hadoop.hive.llap;
 
 import java.io.IOException;
+import java.net.InetAddress;
+import java.net.UnknownHostException;
 import java.security.PrivilegedAction;
 import java.util.HashSet;
 import java.util.Iterator;
@@ -430,6 +432,9 @@ public abstract class AsyncPbRpcProxy nodeToken) {
 String hostId = getHostIdentifier(nodeId.getHostname(), nodeId.getPort());
+if (LOG.isDebugEnabled()) {
+  LOG.debug("Getting host proxies for {}", hostId);
+}
 try {
   return hostProxies.get(hostId, new Callable() {
 @Override
@@ -481,7 +486,16 @@ public abstract class AsyncPbRpcProxy

[hive] branch master updated: HIVE-22831: Add option in HiveStrictManagedMigration to also move tables converted to external living in old WH (Adam Szita, reviewed by Peter Vary)

2020-02-20 Thread szita
This is an automated email from the ASF dual-hosted git repository.

szita pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git


The following commit(s) were added to refs/heads/master by this push:
 new faaf2c3  HIVE-22831: Add option in HiveStrictManagedMigration to also 
move tables converted to external living in old WH (Adam Szita, reviewed by 
Peter Vary)
faaf2c3 is described below

commit faaf2c3fda527cbebf0ed22ab95691ddb9ad8588
Author: Adam Szita 
AuthorDate: Thu Feb 6 17:10:39 2020 +0100

HIVE-22831: Add option in HiveStrictManagedMigration to also move tables 
converted to external living in old WH (Adam Szita, reviewed by Peter Vary)
---
 .../hive/ql/util/HiveStrictManagedMigration.java   | 251 ++---
 .../ql/util/TestHiveStrictManagedMigration.java| 202 -
 2 files changed, 368 insertions(+), 85 deletions(-)

diff --git 
a/ql/src/java/org/apache/hadoop/hive/ql/util/HiveStrictManagedMigration.java 
b/ql/src/java/org/apache/hadoop/hive/ql/util/HiveStrictManagedMigration.java
index 19fb750..e2e5983 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/util/HiveStrictManagedMigration.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/util/HiveStrictManagedMigration.java
@@ -73,6 +73,10 @@ import org.slf4j.LoggerFactory;
 import com.google.common.annotations.VisibleForTesting;
 import com.google.common.collect.Lists;
 
+import static java.util.stream.Collectors.toList;
+import static org.apache.hadoop.hive.metastore.TableType.EXTERNAL_TABLE;
+import static org.apache.hadoop.hive.metastore.TableType.MANAGED_TABLE;
+
 public class HiveStrictManagedMigration {
 
   private static final Logger LOG = 
LoggerFactory.getLogger(HiveStrictManagedMigration.class);
@@ -91,9 +95,10 @@ public class HiveStrictManagedMigration {
 final String oldWarehouseRoot;
 final TableMigrationOption migrationOption;
 final Properties confProps;
-final boolean shouldModifyManagedTableLocation;
+boolean shouldModifyManagedTableLocation;
 final boolean shouldModifyManagedTableOwner;
 final boolean shouldModifyManagedTablePermissions;
+boolean shouldMoveExternal;
 final boolean dryRun;
 final TableType tableType;
 final int tablePoolSize;
@@ -106,6 +111,7 @@ public class HiveStrictManagedMigration {
boolean shouldModifyManagedTableLocation,
boolean shouldModifyManagedTableOwner,
boolean shouldModifyManagedTablePermissions,
+   boolean shouldMoveExternal,
boolean dryRun,
TableType tableType,
int tablePoolSize) {
@@ -118,24 +124,18 @@ public class HiveStrictManagedMigration {
   this.shouldModifyManagedTableLocation = shouldModifyManagedTableLocation;
   this.shouldModifyManagedTableOwner = shouldModifyManagedTableOwner;
   this.shouldModifyManagedTablePermissions = 
shouldModifyManagedTablePermissions;
+  this.shouldMoveExternal = shouldMoveExternal;
   this.dryRun = dryRun;
   this.tableType = tableType;
   this.tablePoolSize = tablePoolSize;
 }
 
-public RunOptions setShouldModifyManagedTableLocation(boolean 
shouldModifyManagedTableLocation) {
-  return new RunOptions(
-  this.dbRegex,
-  this.tableRegex,
-  this.oldWarehouseRoot,
-  this.migrationOption,
-  this.confProps,
-  shouldModifyManagedTableLocation,
-  this.shouldModifyManagedTableOwner,
-  this.shouldModifyManagedTablePermissions,
-  this.dryRun,
-  this.tableType,
-  this.tablePoolSize);
+public void setShouldModifyManagedTableLocation(boolean 
shouldModifyManagedTableLocation) {
+  this.shouldModifyManagedTableLocation = shouldModifyManagedTableLocation;
+}
+
+public void setShouldMoveExternal(boolean shouldMoveExternal) {
+  this.shouldMoveExternal = shouldMoveExternal;
 }
 
 @Override
@@ -149,6 +149,7 @@ public class HiveStrictManagedMigration {
   ", shouldModifyManagedTableLocation=" + 
shouldModifyManagedTableLocation +
   ", shouldModifyManagedTableOwner=" + 
shouldModifyManagedTableOwner +
   ", shouldModifyManagedTablePermissions=" + 
shouldModifyManagedTablePermissions +
+  ", shouldMoveExternal=" + shouldMoveExternal +
   ", dryRun=" + dryRun +
   ", tableType=" + tableType +
   ", tablePoolSize=" + tablePoolSize +
@@ -172,16 +173,22 @@ public class HiveStrictManagedMigration {
 
   private static class WarehouseRootCheckResult {
 final boolean shouldModifyManagedTableLocation;
-final Path curWhRootPath;
+final boolean shouldMoveExternal;
+final Path targetPath;
 final HadoopShims.HdfsEncryptionShim encryptionShim;
+final HadoopShims.HdfsErasureCodingShim ecShim;
 
 WarehouseRootCheckResult(
 boolean 

[hive] branch master updated (fd08239 -> 2705e93)

2020-02-20 Thread mgergely
This is an automated email from the ASF dual-hosted git repository.

mgergely pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git.


from fd08239  HIVE-22881: Revise non-recommended Calcite api calls (Zoltan 
Haindrich reviewed by Jesus Camacho Rodriguez)
 add 2705e93  HIVE-22897 Remove enforcing of package-info.java files from 
the rest of the checkstyle files (Miklos Gergely, reviewed by Peter Vary)

No new revisions were added by this update.

Summary of changes:
 standalone-metastore/checkstyle/checkstyle.xml | 3 ---
 storage-api/checkstyle/checkstyle.xml  | 3 ---
 2 files changed, 6 deletions(-)