[ https://issues.apache.org/jira/browse/HADOOP-9421?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13690496#comment-13690496 ]
Luke Lu commented on HADOOP-9421: --------------------------------- bq. Ok, so now how would you handle improved serverId based tokens? Always take the REINITIATE hit? Maintain a persistent (serverId -> mech) cache in /tmp/hadoop-$user/rpc. Again: it should be choice of protocol users (system designers). bq. In this case, can't the client just blast the INITIATE before getting a NEGOTIATE? Like I said before, ignoring NEGOTIATE in client increase client complexity to ignore the right NEGOTIATE. What if server upgrade/security fix happened in between? bq. do you believe there will actually be a measurable performance difference? Depending on the use case, e.g. a console managing multiple hadoop clusters in multiple regions: the additional round trip could be in hundreds of ms! which is extremely annoying. bq. Will having the client ignore the inflight NEGOTIATE on a reconnect have a measurable latency Like I said, client will have to decide when to ignore the NEGOTIATE, which increase client complexity while still feel dirty. I've shown that this ugliness is not necessary. bq. If so, is the extra round trip for REINITIATEs a bad thing REINITIATE is only supposed to happen when mech cache goes stale. bq. I'll just add it. I'll re-leverage the existing states to avoid adding the reinitate. IMO, explicit REINITIATE make things a lot easier to reason about. bq. prevent a client seizing a connection thread. What're you talking about? The new protocol allows completely non-blocking client. > Convert SASL to use ProtoBuf and add lengths for non-blocking processing > ------------------------------------------------------------------------ > > Key: HADOOP-9421 > URL: https://issues.apache.org/jira/browse/HADOOP-9421 > Project: Hadoop Common > Issue Type: Sub-task > Affects Versions: 2.0.3-alpha > Reporter: Sanjay Radia > Assignee: Daryn Sharp > Priority: Blocker > Attachments: HADOOP-9421.patch, HADOOP-9421.patch, HADOOP-9421.patch, > HADOOP-9421.patch, HADOOP-9421.patch, HADOOP-9421.patch, HADOOP-9421.patch, > HADOOP-9421.patch, HADOOP-9421-v2-demo.patch > > -- 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