You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Zoltan Haindrich <ki...@rxd.hu> on 2017/04/13 19:42:29 UTC
Review Request 58432: HIVE-16416 Service: move constants out from HiveAuthFactory

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

Review request for hive, Thejas Nair and Vaibhav Gumashta.


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


Repository: hive-git


Description
-------

this is a minor change which enables relaxes the requirement of the whole HiveAuthFactory for the client codes to function.

I've moved the constants into a new class


Diffs
-----

  beeline/src/test/org/apache/hive/beeline/ProxyAuthTest.java 0a08389727a5f27aaa78a2e210baad75b949c89f 
  itests/hive-minikdc/src/test/java/org/apache/hive/minikdc/TestHiveAuthFactory.java 0405ff3cb282201ec2c8a39fbddf965bd1acb3d8 
  itests/hive-minikdc/src/test/java/org/apache/hive/minikdc/TestJdbcWithMiniKdc.java daf0f7e6ff199760ecbb6fe16c189b3efed5a64a 
  itests/hive-unit/src/test/java/org/apache/hive/service/cli/thrift/TestThriftCLIServiceWithBinary.java ceaf0e9269590bbbe2c89aa54a687fc4a040273a 
  itests/hive-unit/src/test/java/org/apache/hive/service/cli/thrift/TestThriftCLIServiceWithHttp.java b1c6b6744e567ee2d429b93016cc276e9d27cf30 
  itests/hive-unit/src/test/java/org/apache/hive/service/cli/thrift/TestThriftHttpCLIServiceFeatures.java 1581f9b721a7689c2c64399e5faced7754ceb340 
  itests/hive-unit/src/test/java/org/apache/hive/service/cli/thrift/ThriftCliServiceMessageSizeTest.java 71483073decbbd862ac4be3ceaae12588da56479 
  jdbc/src/java/org/apache/hive/jdbc/HiveConnection.java fb18adb69232de9e595611cb95c025378d04f9fb 
  service/src/java/org/apache/hive/service/auth/HiveAuthConstants.java PRE-CREATION 
  service/src/java/org/apache/hive/service/auth/HiveAuthFactory.java dcb6338d3977665b3c3e954acc5cd8dce4ac5048 
  service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java 0fdc8d90746143c3a81f6bb4e9eaa0f123a12400 
  service/src/java/org/apache/hive/service/cli/thrift/ThriftHttpServlet.java fbe6da40201fd178501d17e58f813616d46b9e78 
  service/src/test/org/apache/hive/service/cli/TestRetryingThriftCLIServiceClient.java 79953c484253dde9ddc1cfaf6616372dd5e564ac 
  service/src/test/org/apache/hive/service/cli/session/TestSessionHooks.java 2878825d93c1271dfdc046fe2c31004559a91d81 
  service/src/test/org/apache/hive/service/cli/thrift/ThriftCliServiceTestWithCookie.java 6fec947d629bcecc7a20c5e764243bb837012880 


Diff: https://reviews.apache.org/r/58432/diff/1/


Testing
-------


Thanks,

Zoltan Haindrich