Flume
  1. Flume
  2. FLUME-1756

Avro client should be able to use load balancing RPC

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: v1.4.0
    • Component/s: None
    • Labels:
      None

      Description

      Avro CLI client cannot use load balancing RPC mechanism

      1. flume.avro-mem-logger.properties
        0.4 kB
        Mike Percy
      2. client.simple.properties
        0.8 kB
        Mike Percy
      3. FLUME-1756-4b.patch
        6 kB
        Mike Percy
      4. FLUME-1756-2.patch
        9 kB
        Mike Percy

        Issue Links

          Activity

          Hide
          Hudson added a comment -

          Integrated in flume-trunk #344 (See https://builds.apache.org/job/flume-trunk/344/)
          FLUME-1756. Avro client should be able to use load balancing RPC (Revision 88980d06b0b5b8af632055af104a4a6e03b32a62)

          Result = SUCCESS
          hshreedharan : http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=88980d06b0b5b8af632055af104a4a6e03b32a62
          Files :

          • flume-ng-core/src/main/java/org/apache/flume/client/avro/AvroCLIClient.java
          • bin/flume-ng
          Show
          Hudson added a comment - Integrated in flume-trunk #344 (See https://builds.apache.org/job/flume-trunk/344/ ) FLUME-1756 . Avro client should be able to use load balancing RPC (Revision 88980d06b0b5b8af632055af104a4a6e03b32a62) Result = SUCCESS hshreedharan : http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=88980d06b0b5b8af632055af104a4a6e03b32a62 Files : flume-ng-core/src/main/java/org/apache/flume/client/avro/AvroCLIClient.java bin/flume-ng
          Hide
          Hari Shreedharan added a comment -

          Patch committed, rev: 88980d06b0b5b8af632055af104a4a6e03b32a62. Thanks Mike!

          Show
          Hari Shreedharan added a comment - Patch committed, rev: 88980d06b0b5b8af632055af104a4a6e03b32a62. Thanks Mike!
          Hide
          Mike Percy added a comment -

          Attaching a couple of files I used for manual testing

          Show
          Mike Percy added a comment - Attaching a couple of files I used for manual testing
          Hide
          Mike Percy added a comment -

          Updated patch to address Brock's concerns

          Show
          Mike Percy added a comment - Updated patch to address Brock's concerns
          Hide
          Brock Noland added a comment -

          Mike,

          Did you want to create a RB item for this or did I miss it?

          Feedback:
          long opt rpc-props all the other long opts are camel caps. Perhaps we should keep this?
          Perhaps we should ensure the properties file exists so as to give the users a good error message when it doesn't exist?

          Show
          Brock Noland added a comment - Mike, Did you want to create a RB item for this or did I miss it? Feedback: long opt rpc-props all the other long opts are camel caps. Perhaps we should keep this? Perhaps we should ensure the properties file exists so as to give the users a good error message when it doesn't exist?
          Hide
          Mike Percy added a comment -

          Rough patch, needs a bit of cleanup I think

          Show
          Mike Percy added a comment - Rough patch, needs a bit of cleanup I think

            People

            • Assignee:
              Mike Percy
              Reporter:
              Mike Percy
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development