Re: Review Request 72519: HIVE-23292

2020-05-16 Thread Ashutosh Chauhan

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

(Updated May 16, 2020, 7:08 p.m.)


Review request for hive and Rajesh Balamohan.


Bugs: HIVE-23292
https://issues.apache.org/jira/browse/HIVE-23292


Repository: hive-git


Description
---

dont serialize unneeded params in query plan


Diffs (updated)
-

  
itests/hive-blobstore/src/test/results/clientpositive/insert_into_dynamic_partitions.q.out
 3783c15203 
  itests/hive-blobstore/src/test/results/clientpositive/insert_into_table.q.out 
92c785c15a 
  
itests/hive-blobstore/src/test/results/clientpositive/insert_overwrite_directory.q.out
 a113a225ba 
  
itests/hive-blobstore/src/test/results/clientpositive/insert_overwrite_dynamic_partitions.q.out
 91e95c4213 
  
itests/hive-blobstore/src/test/results/clientpositive/insert_overwrite_table.q.out
 96e77ed625 
  
itests/hive-blobstore/src/test/results/clientpositive/write_final_output_blobstore.q.out
 2dd98ef438 
  ql/src/java/org/apache/hadoop/hive/ql/plan/PartitionDesc.java b226ab7b24 
  ql/src/java/org/apache/hadoop/hive/ql/plan/TableDesc.java 7993779562 
  ql/src/test/results/clientpositive/beeline/smb_mapjoin_11.q.out 40895970a9 
  ql/src/test/results/clientpositive/beeline/smb_mapjoin_12.q.out d7bd71aef8 
  ql/src/test/results/clientpositive/beeline/smb_mapjoin_13.q.out 70033d814c 
  ql/src/test/results/clientpositive/binary_output_format.q.out b414360855 
  ql/src/test/results/clientpositive/bucket_map_join_1.q.out 440345fa53 
  ql/src/test/results/clientpositive/bucket_map_join_2.q.out 20b27033b8 
  ql/src/test/results/clientpositive/bucket_map_join_spark1.q.out 98a45ee2fd 
  ql/src/test/results/clientpositive/bucket_map_join_spark2.q.out 902c1291f3 
  ql/src/test/results/clientpositive/bucket_map_join_spark3.q.out 42a6998e95 
  ql/src/test/results/clientpositive/bucket_map_join_spark4.q.out 5e6a28b4e8 
  ql/src/test/results/clientpositive/bucketcontext_1.q.out 35b6ae89ca 
  ql/src/test/results/clientpositive/bucketcontext_2.q.out 442e93b778 
  ql/src/test/results/clientpositive/bucketcontext_3.q.out 8fa490d68d 
  ql/src/test/results/clientpositive/bucketcontext_4.q.out c0f1017536 
  ql/src/test/results/clientpositive/bucketcontext_5.q.out ac4496cf2f 
  ql/src/test/results/clientpositive/bucketcontext_6.q.out 39ed5cce50 
  ql/src/test/results/clientpositive/bucketcontext_7.q.out eb64514c9e 
  ql/src/test/results/clientpositive/bucketcontext_8.q.out 245b9618ea 
  ql/src/test/results/clientpositive/bucketmapjoin10.q.out c4d607054f 
  ql/src/test/results/clientpositive/bucketmapjoin11.q.out 16114c9549 
  ql/src/test/results/clientpositive/bucketmapjoin12.q.out 5c453948cb 
  ql/src/test/results/clientpositive/bucketmapjoin13.q.out 693377f863 
  ql/src/test/results/clientpositive/bucketmapjoin5.q.out b0e2931570 
  ql/src/test/results/clientpositive/bucketmapjoin8.q.out e1b658e1b4 
  ql/src/test/results/clientpositive/bucketmapjoin9.q.out 709c780fd1 
  ql/src/test/results/clientpositive/bucketmapjoin_negative.q.out c90fa595dc 
  ql/src/test/results/clientpositive/bucketmapjoin_negative2.q.out 1adac2520c 
  ql/src/test/results/clientpositive/bucketmapjoin_negative3.q.out 2958ae11ed 
  ql/src/test/results/clientpositive/druid/druidkafkamini_basic.q.out 
42e9e5279d 
  ql/src/test/results/clientpositive/druid/druidmini_expressions.q.out 
a8af291989 
  
ql/src/test/results/clientpositive/encrypted/encryption_join_unencrypted_tbl.q.out
 d6115d84be 
  
ql/src/test/results/clientpositive/encrypted/encryption_join_with_different_encryption_keys.q.out
 f6c7d50f46 
  ql/src/test/results/clientpositive/erasurecoding/erasure_explain.q.out 
adad89ef02 
  ql/src/test/results/clientpositive/join_map_ppr.q.out e3d8212352 
  ql/src/test/results/clientpositive/kafka/kafka_storage_handler.q.out 
71af39caaa 
  ql/src/test/results/clientpositive/list_bucket_dml_8.q.out 985195850a 
  ql/src/test/results/clientpositive/llap/acid_bucket_pruning.q.out 81859fe57b 
  ql/src/test/results/clientpositive/llap/acid_nullscan.q.out 85d58dddb4 
  ql/src/test/results/clientpositive/llap/alter_partition_coltype.q.out 
afffbd2862 
  ql/src/test/results/clientpositive/llap/analyze_table_null_partition.q.out 
3206980bd1 
  ql/src/test/results/clientpositive/llap/autoColumnStats_5a.q.out 24b40f0632 
  ql/src/test/results/clientpositive/llap/autoColumnStats_8.q.out 3bb474c927 
  ql/src/test/results/clientpositive/llap/auto_join_reordering_values.q.out 
e87dc546a0 
  ql/src/test/results/clientpositive/llap/auto_sortmerge_join_1.q.out 
096e225844 
  ql/src/test/results/clientpositive/llap/auto_sortmerge_join_11.q.out 
4d428a6040 
  ql/src/test/results/clientpositive/llap/auto_sortmerge_join_12.q.out 
cf73803c06 
  ql/src/test/results/clientpositive/llap/auto_sortmerge_join_2.q.out 
60cfb52549 
  ql/src/test/results/clientpositive/llap/auto_sortmerge_join_3.q.out 
a9b2f03

Review Request 72519: HIVE-23292

2020-05-16 Thread Ashutosh Chauhan

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

Review request for hive and Rajesh Balamohan.


Bugs: HIVE-23292
https://issues.apache.org/jira/browse/HIVE-23292


Repository: hive-git


Description
---

dont serialize unneeded params in query plan


Diffs
-

  
itests/hive-blobstore/src/test/results/clientpositive/insert_into_dynamic_partitions.q.out
 3783c15203 
  itests/hive-blobstore/src/test/results/clientpositive/insert_into_table.q.out 
92c785c15a 
  
itests/hive-blobstore/src/test/results/clientpositive/insert_overwrite_directory.q.out
 a113a225ba 
  
itests/hive-blobstore/src/test/results/clientpositive/insert_overwrite_dynamic_partitions.q.out
 91e95c4213 
  
itests/hive-blobstore/src/test/results/clientpositive/insert_overwrite_table.q.out
 96e77ed625 
  
itests/hive-blobstore/src/test/results/clientpositive/write_final_output_blobstore.q.out
 2dd98ef438 
  ql/src/java/org/apache/hadoop/hive/ql/plan/PartitionDesc.java b226ab7b24 
  ql/src/java/org/apache/hadoop/hive/ql/plan/TableDesc.java 7993779562 
  ql/src/test/results/clientpositive/binary_output_format.q.out b414360855 
  ql/src/test/results/clientpositive/bucket_map_join_1.q.out 440345fa53 
  ql/src/test/results/clientpositive/bucket_map_join_2.q.out 20b27033b8 
  ql/src/test/results/clientpositive/bucket_map_join_spark1.q.out 98a45ee2fd 
  ql/src/test/results/clientpositive/bucket_map_join_spark2.q.out 902c1291f3 
  ql/src/test/results/clientpositive/bucket_map_join_spark3.q.out 42a6998e95 
  ql/src/test/results/clientpositive/bucket_map_join_spark4.q.out 5e6a28b4e8 
  ql/src/test/results/clientpositive/bucketcontext_1.q.out 35b6ae89ca 
  ql/src/test/results/clientpositive/bucketcontext_2.q.out 442e93b778 
  ql/src/test/results/clientpositive/bucketcontext_3.q.out 8fa490d68d 
  ql/src/test/results/clientpositive/bucketcontext_4.q.out c0f1017536 
  ql/src/test/results/clientpositive/bucketcontext_5.q.out ac4496cf2f 
  ql/src/test/results/clientpositive/bucketcontext_6.q.out 39ed5cce50 
  ql/src/test/results/clientpositive/bucketcontext_7.q.out eb64514c9e 
  ql/src/test/results/clientpositive/bucketcontext_8.q.out 245b9618ea 
  ql/src/test/results/clientpositive/bucketmapjoin10.q.out c4d607054f 
  ql/src/test/results/clientpositive/bucketmapjoin11.q.out 16114c9549 
  ql/src/test/results/clientpositive/bucketmapjoin12.q.out 5c453948cb 
  ql/src/test/results/clientpositive/bucketmapjoin13.q.out 693377f863 
  ql/src/test/results/clientpositive/bucketmapjoin5.q.out b0e2931570 
  ql/src/test/results/clientpositive/bucketmapjoin8.q.out e1b658e1b4 
  ql/src/test/results/clientpositive/bucketmapjoin9.q.out 709c780fd1 
  ql/src/test/results/clientpositive/bucketmapjoin_negative.q.out c90fa595dc 
  ql/src/test/results/clientpositive/bucketmapjoin_negative2.q.out 1adac2520c 
  ql/src/test/results/clientpositive/bucketmapjoin_negative3.q.out 2958ae11ed 
  
ql/src/test/results/clientpositive/encrypted/encryption_join_unencrypted_tbl.q.out
 d6115d84be 
  ql/src/test/results/clientpositive/join_map_ppr.q.out e3d8212352 
  ql/src/test/results/clientpositive/list_bucket_dml_8.q.out 985195850a 
  ql/src/test/results/clientpositive/llap/autoColumnStats_5a.q.out 24b40f0632 
  ql/src/test/results/clientpositive/llap/autoColumnStats_8.q.out 3bb474c927 
  ql/src/test/results/clientpositive/llap/auto_sortmerge_join_1.q.out 
096e225844 
  ql/src/test/results/clientpositive/llap/auto_sortmerge_join_11.q.out 
4d428a6040 
  ql/src/test/results/clientpositive/llap/auto_sortmerge_join_12.q.out 
cf73803c06 
  ql/src/test/results/clientpositive/llap/auto_sortmerge_join_2.q.out 
60cfb52549 
  ql/src/test/results/clientpositive/llap/auto_sortmerge_join_3.q.out 
a9b2f03838 
  ql/src/test/results/clientpositive/llap/auto_sortmerge_join_4.q.out 
6ef466fa20 
  ql/src/test/results/clientpositive/llap/auto_sortmerge_join_5.q.out 
f9c09d5c41 
  ql/src/test/results/clientpositive/llap/auto_sortmerge_join_7.q.out 
a20ac713bd 
  ql/src/test/results/clientpositive/llap/auto_sortmerge_join_8.q.out 
9b5a8ef36f 
  ql/src/test/results/clientpositive/llap/bucket1.q.out 3ec523f461 
  ql/src/test/results/clientpositive/llap/bucket2.q.out 1c1db2f114 
  ql/src/test/results/clientpositive/llap/bucket3.q.out 3b303bd76b 
  ql/src/test/results/clientpositive/llap/bucket4.q.out b3831e3169 
  ql/src/test/results/clientpositive/llap/bucket_many.q.out b478b64c3a 
  ql/src/test/results/clientpositive/llap/bucket_map_join_tez2.q.out 414b143ce5 
  ql/src/test/results/clientpositive/llap/bucket_num_reducers.q.out ccc177bdae 
  ql/src/test/results/clientpositive/llap/bucket_num_reducers2.q.out 5ea3d6f14e 
  ql/src/test/results/clientpositive/llap/bucketmapjoin1.q.out a5b97e1faa 
  ql/src/test/results/clientpositive/llap/bucketmapjoin2.q.out f5a8dc2dfd 
  ql/src/test/results/clientpositive/llap/bucketmapjoin3.q.out e11bb747be