You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@flume.apache.org by Denes Arvay <de...@cloudera.com> on 2017/04/06 15:12:09 UTC

Pending pull requests

Hi All,

There are a couple of open, mostly already reviewed pull requests.
I'd like to ask the committers to do a final review and if they think the
patches are up to scratch commit them.

- https://github.com/apache/flume/pull/127 - FLUME-3080 Call
DistributedFileSystem.recoverLease() if close() fails to avoid lease leak -
review needed
- https://github.com/apache/flume/pull/126 - doc fix - already
reviewed, +1'd
- https://github.com/apache/flume/pull/125 NPE fix in Kafka Sink & Channel
- already reviewed, +1'd
- https://github.com/apache/flume/pull/121 - FLUME-3072 Add IP address to
headers in flume log4j appender - reviewed, 2 LGTMs
- https://github.com/apache/flume/pull/120 - FLUME-2579 Support durable
subscription in JMSSource - reviewed, lgtm'd
- https://github.com/apache/flume/pull/115 - doc fix - "shipit!"
- https://github.com/apache/flume/pull/113
<https://github.com/apache/flume/pull/113/files> - FLUME-3068 Ignore hidden
files in TaildirMatcher - under review, looks good, newly added unit test
hasn't been reviewed yet
- https://github.com/apache/flume/pull/112 - FLUME-3067 Default JAVA_OPTS
"-Xmx20m" may be conflicted with custom defines - reviewed, LGTM
- https://github.com/apache/flume/pull/108 - FLUME-3056 TestApplication
hangs indefinitely - reviewed, LGTM
- https://github.com/apache/flume/pull/107 - doc fix - under review

Thanks in advance,
Denes