Marton Greber has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/21548 )
Change subject: [java] Use MetastoreConf instead of HiveConf ...................................................................... [java] Use MetastoreConf instead of HiveConf Using the HiveConf class for Metastore configuration has been deprecated for a while. The actual configrations have been deprecated for about ~6 years, this happened around Hive 3.1 release [1]. This change replaces the HiveConf properties usage in TestKuduMetastorePlugin.java. [1] https://github.com/apache/hive/commit/105cc6543051fe5697de754a0f093539bdac59ff#diff-b7bbe8545a21ec7d7e9cfe40ef66444789e332996aaa9e7f1430dbe4822a2c9c Change-Id: If1ee0dcecdbc93fb6982509d96b606047e21e5f3 Reviewed-on: http://gerrit.cloudera.org:8080/21548 Reviewed-by: Abhishek Chennaka <achenn...@cloudera.com> Tested-by: Abhishek Chennaka <achenn...@cloudera.com> Reviewed-by: Wang Xixu <1450306...@qq.com> Tested-by: Marton Greber <greber...@gmail.com> Reviewed-by: Marton Greber <greber...@gmail.com> --- M java/kudu-hive/src/test/java/org/apache/kudu/hive/metastore/TestKuduMetastorePlugin.java 1 file changed, 21 insertions(+), 18 deletions(-) Approvals: Abhishek Chennaka: Looks good to me, approved; Verified Wang Xixu: Looks good to me, but someone else must approve Marton Greber: Looks good to me, but someone else must approve; Verified -- To view, visit http://gerrit.cloudera.org:8080/21548 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: If1ee0dcecdbc93fb6982509d96b606047e21e5f3 Gerrit-Change-Number: 21548 Gerrit-PatchSet: 4 Gerrit-Owner: Zoltan Chovan <zcho...@cloudera.com> Gerrit-Reviewer: Abhishek Chennaka <achenn...@cloudera.com> Gerrit-Reviewer: Attila Bukor <abu...@apache.org> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <greber...@gmail.com> Gerrit-Reviewer: Wang Xixu <1450306...@qq.com> Gerrit-Reviewer: Zoltan Chovan <zcho...@cloudera.com>