Uploaded image for project: 'Flume'
  1. Flume
  2. FLUME-221

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

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 0.9.2
    • Sinks+Sources
    • 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

          Activity

            People

              flume_vibhor Disabled imported user
              brucem Bruce Mitchener
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: