[
https://issues.apache.org/jira/browse/HIVE-2489?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13150622#comment-13150622
]
[email protected] commented on HIVE-2489:
-----------------------------------------------------
bq. On 2011-11-15 15:52:04, Ashutosh Chauhan wrote:
bq. > common/src/java/org/apache/hadoop/hive/conf/HiveConf.java, line 249
bq. > <https://reviews.apache.org/r/2833/diff/1/?file=58385#file58385line249>
bq. >
bq. > Typo. Default value is 9083, not 9803.
oops. I'll fix right now.
bq. On 2011-11-15 15:52:04, Ashutosh Chauhan wrote:
bq. > metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java,
line 3696
bq. > <https://reviews.apache.org/r/2833/diff/1/?file=58386#file58386line3696>
bq. >
bq. > Why are you recreating HiveConf object here ?
Without recreating, TestMetaStoreAuthorization.testMetaStoreAuthorization() is
failed.
I thought there're issues HiveConf initialization which can cause the problem.
(To be honest, I don't know all the underlying issues relating to metastore
tests.)
But this workaround can make the tests pass.
if there's better way, please let me know.
Or, preserve previous interface not to change legacy tests?
while I was making this patch, I wondered that updating legacy tests is out of
HIVE-2489 scope or not.
- iryoung
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2833/#review3261
-----------------------------------------------------------
On 2011-11-15 15:19:49, iryoung jeong wrote:
bq.
bq. -----------------------------------------------------------
bq. This is an automatically generated e-mail. To reply, visit:
bq. https://reviews.apache.org/r/2833/
bq. -----------------------------------------------------------
bq.
bq. (Updated 2011-11-15 15:19:49)
bq.
bq.
bq. Review request for hive and Carl Steinbach.
bq.
bq.
bq. Summary
bq. -------
bq.
bq. There're 3 ways to set port now - using default port, from cli(-p or old
style), HIVE_PORT/METASTORE_PORT system env.
bq. And after this issue is fixed, there'll be another way to configure port.
bq.
bq. So, it's necessary cleaning up these.
bq.
bq. Previous setting port precedence looks like this "default port < system
env < CLI option"
bq. (if higher one is set, then use it)
bq.
bq. and, now it'll be "default port < system env < HiveConf < CLI option"
bq.
bq. IMO, all other settings should be finally collected into a HiveConf
object. So I implemented this way for HiveMetaStore.
bq.
bq. 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)
bq.
bq.
bq. This addresses bug HIVE-2489.
bq. https://issues.apache.org/jira/browse/HIVE-2489
bq.
bq.
bq. Diffs
bq. -----
bq.
bq.
shims/src/test/org/apache/hadoop/hive/thrift/TestHadoop20SAuthBridge.java
aadb34d
bq. common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 2a52fd1
bq. metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java
ffce4fe
bq. metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java
cd533b6
bq.
metastore/src/test/org/apache/hadoop/hive/metastore/TestMetaStoreAuthorization.java
a089269
bq. service/src/java/org/apache/hadoop/hive/service/HiveServer.java a2d599f
bq.
bq. Diff: https://reviews.apache.org/r/2833/diff
bq.
bq.
bq. Testing
bq. -------
bq.
bq. previous tests passed
bq.
bq.
bq. Thanks,
bq.
bq. iryoung
bq.
bq.
> Add configuration properties for HiveServer and HiveMetaStore port numbers
> --------------------------------------------------------------------------
>
> Key: HIVE-2489
> URL: https://issues.apache.org/jira/browse/HIVE-2489
> Project: Hive
> Issue Type: New Feature
> Components: Configuration, Metastore
> Reporter: Carl Steinbach
> Attachments: HIVE-2489.patch
>
>
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
