Uploaded image for project: 'Flume (READ-ONLY)'
  1. Flume (READ-ONLY)
  2. FLUME-221

Consider renaming rpcSink / rpcSource to reflect that they are Thrift specific

          Details

          • Type: Improvement
          • Status: Closed
          • Priority: Major
          • Resolution: Fixed
          • Affects Version/s: None
          • Fix Version/s: v0.9.2
          • Component/s: Sinks+Sources
          • Labels:
            None

            Description

            As support may be added soon for other RPC transports, would it make sense to rename rpcSink / rpcSource to thriftRpcSink / thriftRpcSource? Then the others could be like avroHttpRpcSink and avroBinaryRpcSink and so on.

            I noticed that there is a tSource as an alias but that seems too terse to me to be easily understandable.

              Attachments

                Issue Links

                relates to

                Sub-task - The sub-task of the issue FLUME-138 Port Event Server to Avro

                • Critical - Crashes, loss of data, severe memory leak.
                • Closed

                  Activity

                    People

                    • Assignee:
                      vibhor vibhor bhatt
                      Reporter:
                      bruce.mitchener Bruce Mitchener
                    • Votes:
                      0 Vote for this issue
                      Watchers:
                      0 Start watching this issue

                      Dates

                      • Created:
                        Updated:
                        Resolved: