You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Alexander Kolbasov <ak...@gmail.com> on 2018/09/03 03:15:20 UTC

Re: Review Request 68552: HIVE-20483: Really move metastore common classes into metastore-common

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

(Updated Sept. 3, 2018, 3:15 a.m.)


Review request for hive, Alan Gates, Peter Vary, and Vihang Karajgaonkar.


Changes
-------

Added test dependency on metastore-server for pig hcatalog tests


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


Repository: hive-git


Description
-------

HIVE-20483: Really move metastore common classes into metastore-common


Diffs (updated)
-----

  beeline/pom.xml 4567d5e09b706ba9023cb901dc796389f7ccf586 
  hcatalog/hcatalog-pig-adapter/pom.xml a1c8ddf057c1681f30956a1cd7d564852eca1a2b 
  metastore/pom.xml 7f751a49359b9661e0f54528688a25669973785b 
  ql/pom.xml a55cbe380db5123b6080eb938e3cc248cf5f5038 
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/common/StatsSetupConst.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/ColumnType.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/FileMetadataHandler.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/MetaStoreFS.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/MetadataStore.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/MetastoreTaskThread.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/PartitionExpressionProxy.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/ReplChangeManager.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/RetryingMetaStoreClient.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/TableIterable.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/Warehouse.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/conf/MetastoreConf.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/conf/TimeValidator.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/partition/spec/CompositePartitionSpecProxy.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/partition/spec/PartitionListComposingSpecProxy.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/partition/spec/PartitionSpecProxy.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/partition/spec/PartitionSpecWithSharedSDProxy.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/security/DelegationTokenIdentifier.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/security/DelegationTokenSecretManager.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/security/DelegationTokenSelector.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/security/HadoopThriftAuthBridge.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/security/HadoopThriftAuthBridge23.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/utils/FileUtils.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/utils/HdfsUtils.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/utils/MetaStoreUtils.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/utils/SecurityUtils.java  


Diff: https://reviews.apache.org/r/68552/diff/2/

Changes: https://reviews.apache.org/r/68552/diff/1-2/


Testing
-------


Thanks,

Alexander Kolbasov


Re: Review Request 68552: HIVE-20483: Really move metastore common classes into metastore-common

Posted by Alexander Kolbasov <ak...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68552/
-----------------------------------------------------------

(Updated Sept. 4, 2018, 4:29 a.m.)


Review request for hive, Alan Gates, Peter Vary, and Vihang Karajgaonkar.


Changes
-------

Added streaming test dependency on metastore-server


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


Repository: hive-git


Description
-------

HIVE-20483: Really move metastore common classes into metastore-common


Diffs (updated)
-----

  beeline/pom.xml 4567d5e09b706ba9023cb901dc796389f7ccf586 
  hcatalog/hcatalog-pig-adapter/pom.xml a1c8ddf057c1681f30956a1cd7d564852eca1a2b 
  hcatalog/pom.xml 4894e9a16fb64625ffe9e875b12f0fb2823fc17a 
  metastore/pom.xml 7f751a49359b9661e0f54528688a25669973785b 
  ql/pom.xml a55cbe380db5123b6080eb938e3cc248cf5f5038 
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/common/StatsSetupConst.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/ColumnType.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/FileMetadataHandler.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/MetaStoreFS.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/MetadataStore.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/MetastoreTaskThread.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/PartitionExpressionProxy.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/ReplChangeManager.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/RetryingMetaStoreClient.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/TableIterable.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/Warehouse.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/conf/MetastoreConf.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/conf/TimeValidator.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/partition/spec/CompositePartitionSpecProxy.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/partition/spec/PartitionListComposingSpecProxy.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/partition/spec/PartitionSpecProxy.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/partition/spec/PartitionSpecWithSharedSDProxy.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/security/DelegationTokenIdentifier.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/security/DelegationTokenSecretManager.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/security/DelegationTokenSelector.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/security/HadoopThriftAuthBridge.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/security/HadoopThriftAuthBridge23.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/utils/FileUtils.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/utils/HdfsUtils.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/utils/MetaStoreUtils.java  
  standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/utils/SecurityUtils.java  
  streaming/pom.xml 99697a8575d80f47195613981f48ffb73509eaca 


Diff: https://reviews.apache.org/r/68552/diff/3/

Changes: https://reviews.apache.org/r/68552/diff/2-3/


Testing
-------


Thanks,

Alexander Kolbasov