Flume
  1. Flume
  2. FLUME-1119

Remove default ports for syslog sources

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: v1.2.0
    • Fix Version/s: v1.2.0
    • Component/s: Sinks+Sources
    • Labels:
      None

      Description

      Remove defaults for syslog sources, since flume will need to run as superuser to connect to these, or will need specific workarounds.

      1. FLUME-1119-1.patch
        3 kB
        Hari Shreedharan

        Activity

        Hari Shreedharan created issue -
        Hide
        jiraposter@reviews.apache.org added a comment -

        -----------------------------------------------------------
        This is an automatically generated e-mail. To reply, visit:
        https://reviews.apache.org/r/4695/
        -----------------------------------------------------------

        Review request for Flume.

        Summary
        -------

        Remove default ports for syslog sources

        This addresses bug FLUME-1119.
        https://issues.apache.org/jira/browse/FLUME-1119

        Diffs


        flume-ng-core/src/main/java/org/apache/flume/source/SyslogUDPSource.java 981c0ee
        flume-ng-core/src/main/java/org/apache/flume/source/SyslogTcpSource.java a87e85b

        Diff: https://reviews.apache.org/r/4695/diff

        Testing
        -------

        Thanks,

        Hari

        Show
        jiraposter@reviews.apache.org added a comment - ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4695/ ----------------------------------------------------------- Review request for Flume. Summary ------- Remove default ports for syslog sources This addresses bug FLUME-1119 . https://issues.apache.org/jira/browse/FLUME-1119 Diffs flume-ng-core/src/main/java/org/apache/flume/source/SyslogUDPSource.java 981c0ee flume-ng-core/src/main/java/org/apache/flume/source/SyslogTcpSource.java a87e85b Diff: https://reviews.apache.org/r/4695/diff Testing ------- Thanks, Hari
        Hari Shreedharan made changes -
        Field Original Value New Value
        Attachment FLUME-1119-1.patch [ 12522202 ]
        Hari Shreedharan made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Hide
        jiraposter@reviews.apache.org added a comment -

        -----------------------------------------------------------
        This is an automatically generated e-mail. To reply, visit:
        https://reviews.apache.org/r/4695/#review6849
        -----------------------------------------------------------

        Ship it!

        lgtm

        • Prasad

        On 2012-04-11 03:32:17, Hari Shreedharan wrote:

        -----------------------------------------------------------

        This is an automatically generated e-mail. To reply, visit:

        https://reviews.apache.org/r/4695/

        -----------------------------------------------------------

        (Updated 2012-04-11 03:32:17)

        Review request for Flume.

        Summary

        -------

        Remove default ports for syslog sources

        This addresses bug FLUME-1119.

        https://issues.apache.org/jira/browse/FLUME-1119

        Diffs

        -----

        flume-ng-core/src/main/java/org/apache/flume/source/SyslogUDPSource.java 981c0ee

        flume-ng-core/src/main/java/org/apache/flume/source/SyslogTcpSource.java a87e85b

        Diff: https://reviews.apache.org/r/4695/diff

        Testing

        -------

        Thanks,

        Hari

        Show
        jiraposter@reviews.apache.org added a comment - ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4695/#review6849 ----------------------------------------------------------- Ship it! lgtm Prasad On 2012-04-11 03:32:17, Hari Shreedharan wrote: ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4695/ ----------------------------------------------------------- (Updated 2012-04-11 03:32:17) Review request for Flume. Summary ------- Remove default ports for syslog sources This addresses bug FLUME-1119 . https://issues.apache.org/jira/browse/FLUME-1119 Diffs ----- flume-ng-core/src/main/java/org/apache/flume/source/SyslogUDPSource.java 981c0ee flume-ng-core/src/main/java/org/apache/flume/source/SyslogTcpSource.java a87e85b Diff: https://reviews.apache.org/r/4695/diff Testing ------- Thanks, Hari
        Hide
        Prasad Mujumdar added a comment -

        Patch committed to trunk
        Thanks Hari!

        Show
        Prasad Mujumdar added a comment - Patch committed to trunk Thanks Hari!
        Prasad Mujumdar made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Prasad Mujumdar made changes -
        Fix Version/s v1.2.0 [ 12320243 ]
        Hide
        Hudson added a comment -

        Integrated in flume-trunk #169 (See https://builds.apache.org/job/flume-trunk/169/)
        FLUME-1119: Remove default ports for syslog sources
        (Hari Shreedharan via Prasad Mujumdar) (Revision 1324819)

        Result = SUCCESS
        prasadm : http://svn.apache.org/viewvc/?view=rev&rev=1324819
        Files :

        • /incubator/flume/trunk/flume-ng-core/src/main/java/org/apache/flume/source/SyslogTcpSource.java
        • /incubator/flume/trunk/flume-ng-core/src/main/java/org/apache/flume/source/SyslogUDPSource.java
        Show
        Hudson added a comment - Integrated in flume-trunk #169 (See https://builds.apache.org/job/flume-trunk/169/ ) FLUME-1119 : Remove default ports for syslog sources (Hari Shreedharan via Prasad Mujumdar) (Revision 1324819) Result = SUCCESS prasadm : http://svn.apache.org/viewvc/?view=rev&rev=1324819 Files : /incubator/flume/trunk/flume-ng-core/src/main/java/org/apache/flume/source/SyslogTcpSource.java /incubator/flume/trunk/flume-ng-core/src/main/java/org/apache/flume/source/SyslogUDPSource.java

          People

          • Assignee:
            Hari Shreedharan
            Reporter:
            Hari Shreedharan
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development