hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "iryoung jeong" <>
Subject Review Request: HIVE-2489 Add configuration properties for HiveServer and HiveMetaStore port numbers and fix related unittest
Date Tue, 15 Nov 2011 15:18:52 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for hive.


There're 3 ways to set port now - using default port, from cli(-p or old style), HIVE_PORT/METASTORE_PORT
system env.
And after this issue is fixed, there'll be another way to configure port.

So, it's necessary cleaning up these.

Previous setting port precedence looks like this "default port < system env < CLI option"
(if higher one is set, then use it)

and, now it'll be "default port < system env < HiveConf < CLI option"

IMO, all other settings should be finally collected into a HiveConf object. So I implemented
this way for HiveMetaStore.

But in HiveServer, some configurations are only available through CLI options, so I chose
applying port number from HiveConf to previous CLI object.(which is required minimum code
change, and I thought moving other settings to HiveConf is out of scope of this issue)

This addresses bug HIVE-2489.


  shims/src/test/org/apache/hadoop/hive/thrift/ aadb34d 
  common/src/java/org/apache/hadoop/hive/conf/ 2a52fd1 
  metastore/src/java/org/apache/hadoop/hive/metastore/ ffce4fe 
  metastore/src/java/org/apache/hadoop/hive/metastore/ cd533b6 
  metastore/src/test/org/apache/hadoop/hive/metastore/ a089269

  service/src/java/org/apache/hadoop/hive/service/ a2d599f 



previous tests passed



  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message