Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.5.0
    • Component/s: java
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      SaslSocketTransceiver and SaslSocketServer should be preferred to SocketTransceiver and SocketServer. The former are more optimized and offer the option of security. The latter should be deprecated to discourage a proliferation of implementations.

      We should also add Transceiver and Server factory methods that accept a URI. For "http:" URIs these should use HttpTransceiver and HttpServer. For "avro:" URIs these should use SaslSocketTransceiver and SaslSocketServer. The rpcsend and rpcrecieve tools should use these factories.

      1. AVRO-707.patch
        7 kB
        Doug Cutting

        Issue Links

          Activity

          Hide
          Doug Cutting added a comment -

          Here's a patch for this. Change are:

          • Two new static factory methods that understand "http:" and "avro:" URIs, using SaslSocket for "avro:":
            • Transceiver Ipc.createTransceiver(URI)
            • Server Ipc.createServer(Responder, URI)
          • switches the rpcsend and rpcreceive tools to use the new factory methods
          • deprecates SocketServer and SocketTransceiver in favor of SaslSocketServer and SaslSocketTransceiver
          Show
          Doug Cutting added a comment - Here's a patch for this. Change are: Two new static factory methods that understand "http:" and "avro:" URIs, using SaslSocket for "avro:": Transceiver Ipc.createTransceiver(URI) Server Ipc.createServer(Responder, URI) switches the rpcsend and rpcreceive tools to use the new factory methods deprecates SocketServer and SocketTransceiver in favor of SaslSocketServer and SaslSocketTransceiver
          Hide
          Scott Carey added a comment -

          Patch looks good, but I can't test it at the moment.

          Show
          Scott Carey added a comment - Patch looks good, but I can't test it at the moment.
          Hide
          Doug Cutting added a comment -

          I committed this.

          Show
          Doug Cutting added a comment - I committed this.

            People

            • Assignee:
              Doug Cutting
              Reporter:
              Doug Cutting
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development