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

There's no option to set hostname for HTTPSource

    Details

      Description

      I'd like to request the ability to specify a particular host (and thus a specific NIC) for HTTPSource so that user can choose to listen on one NIC or all NICs. Currently only the port number can be specified, not the hostname. Can we add a 'host' prop to HttpSource?

      1. hn.patch
        6 kB
        Ivan Bogdanov

        Activity

        Hide
        hudson Hudson added a comment -

        Integrated in flume-trunk #405 (See https://builds.apache.org/job/flume-trunk/405/)
        FLUME-1867. Add an option to provide hostname for HTTPSource. (Revision 71df908d05c63095ad12082b9ecf1390ee98a400)

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

        • flume-ng-core/src/test/java/org/apache/flume/source/http/TestHTTPSource.java
        • flume-ng-core/src/main/java/org/apache/flume/source/http/HTTPSourceConfigurationConstants.java
        • flume-ng-core/src/main/java/org/apache/flume/source/http/HTTPSource.java
        Show
        hudson Hudson added a comment - Integrated in flume-trunk #405 (See https://builds.apache.org/job/flume-trunk/405/ ) FLUME-1867 . Add an option to provide hostname for HTTPSource. (Revision 71df908d05c63095ad12082b9ecf1390ee98a400) Result = SUCCESS hshreedharan : http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=71df908d05c63095ad12082b9ecf1390ee98a400 Files : flume-ng-core/src/test/java/org/apache/flume/source/http/TestHTTPSource.java flume-ng-core/src/main/java/org/apache/flume/source/http/HTTPSourceConfigurationConstants.java flume-ng-core/src/main/java/org/apache/flume/source/http/HTTPSource.java
        Hide
        hshreedharan Hari Shreedharan added a comment -

        Patch committed, rev: 71df908. Thanks Ivan!

        Show
        hshreedharan Hari Shreedharan added a comment - Patch committed, rev: 71df908. Thanks Ivan!
        Hide
        hshreedharan Hari Shreedharan added a comment -

        +1. This looks good. I will run tests and commit it.

        Show
        hshreedharan Hari Shreedharan added a comment - +1. This looks good. I will run tests and commit it.

          People

          • Assignee:
            vanya_void Ivan Bogdanov
            Reporter:
            will@cloudera.com Will McQueen
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development