[ https://issues.apache.org/jira/browse/HIVE-2767?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Phabricator updated HIVE-2767: ------------------------------ Attachment: HIVE-2767.D2661.1.patch travis requested code review of "HIVE-2767 [jira] Optionally use framed transport with metastore". Reviewers: JIRA Add support for optionally using the thrift framed transport, enabling integration with environments where that is necessary. Users may want/need to use thrift's framed transport when communicating with the Hive MetaStore. This patch adds a new property hive.metastore.thrift.framed.transport.enabled that enables the framed transport (defaults to off, aka no change from before the patch). This property must be set for both clients and the HMS server. It wasn't immediately clear how to use the framed transport with SASL, so as written an exception is thrown if you try starting the server with both options. If SASL and the framed transport will indeed work together I can update the patch (although I don't have a secured environment to test in). TEST PLAN Tested locally that client and server can connect, both with and without the flag. Tests pass. REVISION DETAIL https://reviews.facebook.net/D2661 AFFECTED FILES common/src/java/org/apache/hadoop/hive/conf/HiveConf.java metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java MANAGE HERALD DIFFERENTIAL RULES https://reviews.facebook.net/herald/view/differential/ WHY DID I GET THIS EMAIL? https://reviews.facebook.net/herald/transcript/6111/ Tip: use the X-Herald-Rules header to filter Herald messages in your client. > Optionally use framed transport with metastore > ---------------------------------------------- > > Key: HIVE-2767 > URL: https://issues.apache.org/jira/browse/HIVE-2767 > Project: Hive > Issue Type: New Feature > Components: Metastore > Reporter: Travis Crawford > Assignee: Julien Le Dem > Attachments: HIVE-2767.D2661.1.patch, HIVE-2767.patch.txt, > HIVE-2767_a.patch.txt > > > Users may want/need to use thrift's framed transport when communicating with > the Hive MetaStore. This patch adds a new property > {{hive.metastore.thrift.framed.transport.enabled}} that enables the framed > transport (defaults to off, aka no change from before the patch). This > property must be set for both clients and the HMS server. > It wasn't immediately clear how to use the framed transport with SASL, so as > written an exception is thrown if you try starting the server with both > options. If SASL and the framed transport will indeed work together I can > update the patch (although I don't have a secured environment to test in). -- 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