Return-Path: X-Original-To: apmail-hadoop-common-issues-archive@minotaur.apache.org Delivered-To: apmail-hadoop-common-issues-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 4809110AC2 for ; Tue, 6 Aug 2013 16:36:57 +0000 (UTC) Received: (qmail 82502 invoked by uid 500); 6 Aug 2013 16:36:53 -0000 Delivered-To: apmail-hadoop-common-issues-archive@hadoop.apache.org Received: (qmail 82205 invoked by uid 500); 6 Aug 2013 16:36:51 -0000 Mailing-List: contact common-issues-help@hadoop.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: common-issues@hadoop.apache.org Delivered-To: mailing list common-issues@hadoop.apache.org Received: (qmail 82126 invoked by uid 99); 6 Aug 2013 16:36:49 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 06 Aug 2013 16:36:49 +0000 Date: Tue, 6 Aug 2013 16:36:49 +0000 (UTC) From: "Daryn Sharp (JIRA)" To: common-issues@hadoop.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Commented] (HADOOP-9820) RPCv9 wire protocol is insufficient to support multiplexing MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ https://issues.apache.org/jira/browse/HADOOP-9820?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13730917#comment-13730917 ] Daryn Sharp commented on HADOOP-9820: ------------------------------------- Summary of offline messages to Sanjay who is reviewing: # Sasl wrapped message are sent in RPC/SASL protobufs like other SASL messages # Code shifted in Server to decode SASL wrapped packets in the code path invoked by processRpcOutOfBandRequest which handles other SASL packets # Client's SaslInputStream replacement unwraps SASL wrapped messages, leaves others alone # Client's SaslOutputStream replacement adds the RPC header because existing one does length/encrypted-payload only # Replacement SaslOutputStream correctly uses a buffered stream of the SASL negotiated size for wrapping. Existing SaslOutputStream impl was wrong but accidentally worked because of a smaller buffered stream atop it. # Slight optimization that Client isn't unnecessarily given sasl streams (that are no-ops) when wrapping isn't being done # Per comments, would be cleaner to decode all RPC packets in Client and route SASL messages to SaslRpcClient, but decoding is currently split across Client/SaslRpcClient. SaslRpcClient handles RPC decoding during authentication, but then Client decodes the rest of the stream with no knowledge of SASL. In the future, Client should decode all RPC packets and route SASL to SaslRpcClient. > RPCv9 wire protocol is insufficient to support multiplexing > ----------------------------------------------------------- > > Key: HADOOP-9820 > URL: https://issues.apache.org/jira/browse/HADOOP-9820 > Project: Hadoop Common > Issue Type: Bug > Components: ipc, security > Affects Versions: 3.0.0, 2.1.0-beta > Reporter: Daryn Sharp > Assignee: Daryn Sharp > Priority: Blocker > Attachments: HADOOP-9820.patch > > > RPCv9 is intended to allow future support of multiplexing. This requires all wire messages to be tagged with a RPC header so a demux can decode and route the messages accordingly. > RPC ping packets and SASL QOP wrapped data is known to not be tagged with a header. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira