-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/71763/#review218624
-----------------------------------------------------------




jdbc/src/java/org/apache/hive/jdbc/HiveDriver.java
Line 46 (original), 46 (patched)
<https://reviews.apache.org/r/71763/#comment306420>

    are you sure about this? seems like it can crash the driver, that is not 
the old way of doing thing


- Slim Bouguerra


On Nov. 13, 2019, 5:22 p.m., David Mollitor wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/71763/
> -----------------------------------------------------------
> 
> (Updated Nov. 13, 2019, 5:22 p.m.)
> 
> 
> Review request for hive and Peter Vary.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> HIVE-22484: Remove Calls to printStackTrace
> 
> 
> Diffs
> -----
> 
>   jdbc/src/java/org/apache/hive/jdbc/HiveBaseResultSet.java dfaa40fe23 
>   jdbc/src/java/org/apache/hive/jdbc/HiveDriver.java 102683ee18 
>   ql/src/java/org/apache/hadoop/hive/ql/QueryPlan.java 0d7b92d649 
>   ql/src/java/org/apache/hadoop/hive/ql/ddl/DDLSemanticAnalyzerFactory.java 
> c8aaec15d4 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/FileSinkOperator.java 9ad4e71482 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/HashTableDummyOperator.java 
> e8f7dd067e 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/JoinUtil.java 1aae142ba7 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/ScriptOperator.java 3210ca5cf8 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java a7770b4e53 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/mr/ExecDriver.java cd4f2a02a3 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/tez/CustomPartitionVertex.java 
> dfabfb81e5 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/tez/DagUtils.java 077c94f82b 
>   ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryViewer.java 
> 616f2d6c10 
>   ql/src/java/org/apache/hadoop/hive/ql/io/AcidUtils.java 67996c6db9 
>   ql/src/java/org/apache/hadoop/hive/ql/io/RCFile.java 3e45e45b27 
>   
> ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/LocalMapJoinProcFactory.java
>  3e81ab5959 
>   
> ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/SortMergeJoinTaskDispatcher.java
>  fbf6852013 
>   serde/src/java/org/apache/hadoop/hive/serde2/dynamic_type/DynamicSerDe.java 
> 948cddcb28 
>   
> serde/src/java/org/apache/hadoop/hive/serde2/dynamic_type/DynamicSerDeTypeMap.java
>  3f086cdde4 
>   
> serde/src/java/org/apache/hadoop/hive/serde2/dynamic_type/DynamicSerDeTypeSet.java
>  f41959b7d2 
>   service/src/java/org/apache/hive/service/cli/operation/SQLOperation.java 
> b87b670652 
> 
> 
> Diff: https://reviews.apache.org/r/71763/diff/1/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> David Mollitor
> 
>

Reply via email to