Re: [PR] [FLUME-3480] remove log4j 1.2.17 due to eol [flume]

2023-11-18 Thread via GitHub
rgoers closed pull request #418: [FLUME-3480] remove log4j 1.2.17 due to eol URL: https://github.com/apache/flume/pull/418 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] [FLUME-3480] remove log4j 1.2.17 due to eol [flume]

2023-11-18 Thread via GitHub
rgoers commented on PR #418: URL: https://github.com/apache/flume/pull/418#issuecomment-1817748485 This PR says log4j 1 is used in the build-support module but the PR doesn't show that. The Log4j 1 Appender should simmply be removed. Log4j 2 already provides a Flume Appender and now that

[PR] [FLUME-3480] remove log4j 1.2.17 due to eol [flume]

2023-11-09 Thread via GitHub
nikita15p opened a new pull request, #418: URL: https://github.com/apache/flume/pull/418 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe,

[PR] Bump org.elasticsearch:elasticsearch from 0.90.1 to 7.17.13 [flume-search]

2023-10-30 Thread via GitHub
dependabot[bot] opened a new pull request, #5: URL: https://github.com/apache/flume-search/pull/5 Bumps [org.elasticsearch:elasticsearch](https://github.com/elastic/elasticsearch) from 0.90.1 to 7.17.13. Release notes Sourced from

[GitHub] [flume] SamBhandary opened a new pull request, #417: Calculate batchElapsedTime correctly by converting startTime from nan…

2023-09-07 Thread via GitHub
SamBhandary opened a new pull request, #417: URL: https://github.com/apache/flume/pull/417 ISSUE: batchElapsedTime calculated incorrectly. [DEBUG - org.apache.flume.channel.kafka.ChannelCallback.onCompletion(KafkaChannel.java:760)] Acked message_no 793: test-3-280448-1667034684768

[GitHub] [flume] rgoers commented on pull request #286: FLUME-3216.Avoid reading content repeatedly.

2023-08-10 Thread via GitHub
rgoers commented on PR #286: URL: https://github.com/apache/flume/pull/286#issuecomment-1673457500 I am wondering why this was closed? Is the fix not valid. It would have been nice to have a test to validate this. -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [flume] highfei2011 closed pull request #286: FLUME-3216.Avoid reading content repeatedly.

2023-08-08 Thread via GitHub
highfei2011 closed pull request #286: FLUME-3216.Avoid reading content repeatedly. URL: https://github.com/apache/flume/pull/286 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [flume] nikita15p commented on pull request #411: [FLUME-3473] fixed testMinimumRequiredSpaceTooSmallforPut test by changing buffer size dynamic

2023-07-30 Thread via GitHub
nikita15p commented on PR #411: URL: https://github.com/apache/flume/pull/411#issuecomment-1657442355 @rgoers please review it -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [flume] whitelilis closed pull request #62: bugfix: when log rotate, taildir source will treat it as new file, so…

2023-06-28 Thread via GitHub
whitelilis closed pull request #62: bugfix: when log rotate, taildir source will treat it as new file, so… URL: https://github.com/apache/flume/pull/62 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [flume] dependabot[bot] opened a new pull request, #416: Bump snappy-java from 1.1.8.4 to 1.1.10.1 in /flume-parent

2023-06-15 Thread via GitHub
dependabot[bot] opened a new pull request, #416: URL: https://github.com/apache/flume/pull/416 Bumps [snappy-java](https://github.com/xerial/snappy-java) from 1.1.8.4 to 1.1.10.1. Release notes Sourced from https://github.com/xerial/snappy-java/releases;>snappy-java's releases.

[GitHub] [flume] dependabot[bot] opened a new pull request, #415: Bump guava from 29.0-jre to 32.0.0-jre in /flume-parent

2023-06-14 Thread via GitHub
dependabot[bot] opened a new pull request, #415: URL: https://github.com/apache/flume/pull/415 Bumps [guava](https://github.com/google/guava) from 29.0-jre to 32.0.0-jre. Release notes Sourced from https://github.com/google/guava/releases;>guava's releases. 32.0.0 Maven

[GitHub] [flume-hadoop] dependabot[bot] closed pull request #2: Bump guava from 23.6-jre to 29.0-jre

2023-06-14 Thread via GitHub
dependabot[bot] closed pull request #2: Bump guava from 23.6-jre to 29.0-jre URL: https://github.com/apache/flume-hadoop/pull/2 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment.

[GitHub] [flume-hadoop] dependabot[bot] commented on pull request #2: Bump guava from 23.6-jre to 29.0-jre

2023-06-14 Thread via GitHub
dependabot[bot] commented on PR #2: URL: https://github.com/apache/flume-hadoop/pull/2#issuecomment-1592065615 Superseded by #3. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [flume-hadoop] dependabot[bot] opened a new pull request, #3: Bump guava from 23.6-jre to 32.0.0-jre

2023-06-14 Thread via GitHub
dependabot[bot] opened a new pull request, #3: URL: https://github.com/apache/flume-hadoop/pull/3 Bumps [guava](https://github.com/google/guava) from 23.6-jre to 32.0.0-jre. Release notes Sourced from https://github.com/google/guava/releases;>guava's releases. 32.0.0

[GitHub] [flume] nikita15p commented on pull request #405: [FLUME-3468] fix default value of passwordFile property when userName is set for jms source

2023-06-11 Thread via GitHub
nikita15p commented on PR #405: URL: https://github.com/apache/flume/pull/405#issuecomment-1586554737 @rgoers can u please review this -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [flume] yangjf2019 commented on pull request #409: add year 2023 to Notice files

2023-06-08 Thread via GitHub
yangjf2019 commented on PR #409: URL: https://github.com/apache/flume/pull/409#issuecomment-1583861240 @busbey Done, thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [flume] busbey commented on pull request #409: add year 2023 to Notice files

2023-06-05 Thread via GitHub
busbey commented on PR #409: URL: https://github.com/apache/flume/pull/409#issuecomment-1577028193 I'm -0 on switching to use "Present". At least my reading of [the published ASF policy on NOTICE files](https://www.apache.org/legal/src-headers.html#notice) requires actual years.

[GitHub] [flume] dependabot[bot] commented on pull request #407: Bump jetty-server from 9.4.48.v20220622 to 10.0.14 in /flume-parent

2023-06-04 Thread via GitHub
dependabot[bot] commented on PR #407: URL: https://github.com/apache/flume/pull/407#issuecomment-1575666374 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me

[GitHub] [flume] rgoers closed pull request #407: Bump jetty-server from 9.4.48.v20220622 to 10.0.14 in /flume-parent

2023-06-04 Thread via GitHub
rgoers closed pull request #407: Bump jetty-server from 9.4.48.v20220622 to 10.0.14 in /flume-parent URL: https://github.com/apache/flume/pull/407 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [flume] rgoers merged pull request #414: Bump guava from 23.6-jre to 29.0-jre in /flume-parent

2023-06-04 Thread via GitHub
rgoers merged PR #414: URL: https://github.com/apache/flume/pull/414 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@flume.apache.org

[GitHub] [flume] rgoers merged pull request #412: [FLUME_3476] fixing vulnerability CVE-2023-26048

2023-06-04 Thread via GitHub
rgoers merged PR #412: URL: https://github.com/apache/flume/pull/412 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@flume.apache.org

[GitHub] [flume] dependabot[bot] opened a new pull request, #414: Bump guava from 23.6-jre to 29.0-jre in /flume-parent

2023-06-04 Thread via GitHub
dependabot[bot] opened a new pull request, #414: URL: https://github.com/apache/flume/pull/414 Bumps [guava](https://github.com/google/guava) from 23.6-jre to 29.0-jre. Release notes Sourced from https://github.com/google/guava/releases;>guava's releases. 29.0 Maven

[GitHub] [flume] rgoers merged pull request #413: [FLUME-3477] uptick avro-ipc to 1.11.1

2023-06-04 Thread via GitHub
rgoers merged PR #413: URL: https://github.com/apache/flume/pull/413 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@flume.apache.org

[GitHub] [flume] nikita15p opened a new pull request, #413: [FLUME-3477] uptick avro-ipc to 1.11.1

2023-05-29 Thread via GitHub
nikita15p opened a new pull request, #413: URL: https://github.com/apache/flume/pull/413 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe,

[GitHub] [flume] nikita15p opened a new pull request, #412: [FLUME_3476] fixing vulnerability CVE-2023-26048

2023-05-24 Thread via GitHub
nikita15p opened a new pull request, #412: URL: https://github.com/apache/flume/pull/412 Versions affected are for this CVE: [9.4.10.RC0,9.4.50.v20221201] [10.0.0-alpha0,10.0.0.alpha1] -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [flume] nikita15p opened a new pull request, #411: [FLUME-3473] fixed testMinimumRequiredSpaceTooSmallforPut test by changing buffer size dynamic

2023-05-21 Thread via GitHub
nikita15p opened a new pull request, #411: URL: https://github.com/apache/flume/pull/411 Current state: Since the buffer size is limited by the size ``` byte[] buffer = new byte[64 * 1024]; ``` This was causing while loop to execute limited by buffer size. This was causing

[GitHub] [flume] yangjf2019 commented on a diff in pull request #409: add year 2023 to Notice files

2023-05-15 Thread via GitHub
yangjf2019 commented on code in PR #409: URL: https://github.com/apache/flume/pull/409#discussion_r1194535354 ## flume-ng-dist/src/main/appended-resources/META-INF/NOTICE: ## @@ -217,8 +217,8 @@ source: https://www.opensource.apple.com/source/configd/configd-453.19/dnsinfo/d

[GitHub] [flume] yangjf2019 commented on pull request #409: add year 2023 to Notice files

2023-05-15 Thread via GitHub
yangjf2019 commented on PR #409: URL: https://github.com/apache/flume/pull/409#issuecomment-1548875819 > flume-ng-clients/flume-ng-log4jappender/src/main/resources-for-jar-with-deps/META-INF/NOTICE Thanks for your review.I updated. -- This is an automated message from the Apache

[GitHub] [flume] busbey commented on a diff in pull request #409: add year 2023 to Notice files

2023-05-15 Thread via GitHub
busbey commented on code in PR #409: URL: https://github.com/apache/flume/pull/409#discussion_r1193946950 ## flume-ng-dist/src/main/appended-resources/META-INF/NOTICE: ## @@ -217,8 +217,8 @@ source: https://www.opensource.apple.com/source/configd/configd-453.19/dnsinfo/d

[GitHub] [flume] nikita15p commented on pull request #409: add year 2023 to Notice files

2023-05-15 Thread via GitHub
nikita15p commented on PR #409: URL: https://github.com/apache/flume/pull/409#issuecomment-1547411871 Looks good to me. Cc @busbey @rgoers can u please approve -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [flume-jdbc] deepak-garg commented on pull request #1: FLUME-3475 - Upgrade commons-dbcp to commons-dbcp2

2023-05-12 Thread via GitHub
deepak-garg commented on PR #1: URL: https://github.com/apache/flume-jdbc/pull/1#issuecomment-1546021935 @rgoers -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[GitHub] [flume-jdbc] deepak-garg opened a new pull request, #1: FLUME-3475 - Upgrade commons-dbcp to commons-dbcp2

2023-05-12 Thread via GitHub
deepak-garg opened a new pull request, #1: URL: https://github.com/apache/flume-jdbc/pull/1 https://issues.apache.org/jira/browse/FLUME-3475 -> Please refer above ticket for the logs after this change. Specific testcases related to jdbc-channel and all the testcases for the flume

[GitHub] [flume] changeAtLater opened a new pull request, #410: fix JSONEvent charset bug

2023-05-11 Thread via GitHub
changeAtLater opened a new pull request, #410: URL: https://github.com/apache/flume/pull/410 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe,

[GitHub] [flume-site] rgoers closed pull request #3: [FLUME-3461] Fix flume userguide rendering

2023-04-30 Thread via GitHub
rgoers closed pull request #3: [FLUME-3461] Fix flume userguide rendering URL: https://github.com/apache/flume-site/pull/3 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[GitHub] [flume-site] rgoers commented on pull request #3: [FLUME-3461] Fix flume userguide rendering

2023-04-30 Thread via GitHub
rgoers commented on PR #3: URL: https://github.com/apache/flume-site/pull/3#issuecomment-1529132156 The fix was merged. However, the html file should not have been updated as it gets generated. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [flume] nikita15p opened a new pull request, #408: [FLUME-3472] fix vulnerability CVE 2020-1938 caused by tomcat-embed-core

2023-04-27 Thread via GitHub
nikita15p opened a new pull request, #408: URL: https://github.com/apache/flume/pull/408 org.apache.thrift:libthrift:0.14.2 has dependency on tomcat-embed-core : 8.5.46 which is causing CVE 2020-1938. So added tomcat-embed-core. Maintaining same version of tomcat-embed-el ,

[GitHub] [flume-kafka] rgoers merged pull request #1: FLUME-3470, Upgrade kafka clients jar to version 3.4.0

2023-04-23 Thread via GitHub
rgoers merged PR #1: URL: https://github.com/apache/flume-kafka/pull/1 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@flume.apache.org

[GitHub] [flume] deepak-garg commented on pull request #406: FLUME-3469, Upgrade jackson core and jackson databind jar

2023-04-20 Thread via GitHub
deepak-garg commented on PR #406: URL: https://github.com/apache/flume/pull/406#issuecomment-1516693657 @rgoers looking forward to hear from you on this -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [flume-site] nikita15p opened a new pull request, #3: [FLUME-3461] Fix flume userguide rendering

2023-04-19 Thread via GitHub
nikita15p opened a new pull request, #3: URL: https://github.com/apache/flume-site/pull/3 With this change, the table gets rendered correctly -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [flume] deepak-garg commented on pull request #406: FLUME-3469, Upgrade jackson core and jackson databind jar

2023-04-19 Thread via GitHub
deepak-garg commented on PR #406: URL: https://github.com/apache/flume/pull/406#issuecomment-1514659886 @rgoers, How I can trigger the build for this change -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [flume-kafka] rgoers commented on pull request #1: FLUME-3470, Upgrade kafka clients jar to version 3.4.0

2023-04-18 Thread via GitHub
rgoers commented on PR #1: URL: https://github.com/apache/flume-kafka/pull/1#issuecomment-1514086324 I am sure this will be fine but won't get to it for a couple of days. This component is one of several that are awaiting a release. -- This is an automated message from the Apache Git

[GitHub] [flume] dependabot[bot] opened a new pull request, #407: Bump jetty-server from 9.4.48.v20220622 to 10.0.14 in /flume-parent

2023-04-18 Thread via GitHub
dependabot[bot] opened a new pull request, #407: URL: https://github.com/apache/flume/pull/407 Bumps [jetty-server](https://github.com/eclipse/jetty.project) from 9.4.48.v20220622 to 10.0.14. Release notes Sourced from

[GitHub] [flume-kafka] deepak-garg opened a new pull request, #1: FLUME-3470, Upgrade kafka clients jar to version 3.4.0

2023-04-18 Thread via GitHub
deepak-garg opened a new pull request, #1: URL: https://github.com/apache/flume-kafka/pull/1 https://issues.apache.org/jira/browse/FLUME-3470 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [flume] nikita15p opened a new pull request, #405: [FLUME-3468] fix default value of passwordFile property when userName is set for jms source

2023-04-08 Thread via GitHub
nikita15p opened a new pull request, #405: URL: https://github.com/apache/flume/pull/405 In jms source, when passwordFile property in agent config is not set, currently null value is getting passed which causes exception due to `password = Optional.absent();` in

[GitHub] [flume-hadoop] dependabot[bot] opened a new pull request, #2: Bump guava from 23.6-jre to 29.0-jre

2023-04-06 Thread via GitHub
dependabot[bot] opened a new pull request, #2: URL: https://github.com/apache/flume-hadoop/pull/2 Bumps [guava](https://github.com/google/guava) from 23.6-jre to 29.0-jre. Release notes Sourced from https://github.com/google/guava/releases;>guava's releases. 29.0 Maven

[GitHub] [flume-hadoop] dependabot[bot] opened a new pull request, #1: Bump hadoop-common from 3.2.2 to 3.2.4

2023-04-06 Thread via GitHub
dependabot[bot] opened a new pull request, #1: URL: https://github.com/apache/flume-hadoop/pull/1 Bumps hadoop-common from 3.2.2 to 3.2.4. [![Dependabot compatibility

[GitHub] [flume] rgoers commented on pull request #402: [FLUME-3461] Fix vulnerability in netty-codec-haproxy

2023-04-01 Thread via GitHub
rgoers commented on PR #402: URL: https://github.com/apache/flume/pull/402#issuecomment-1493222941 Verified this locally -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[GitHub] [flume] rgoers merged pull request #402: [FLUME-3461] Fix vulnerability in netty-codec-haproxy

2023-04-01 Thread via GitHub
rgoers merged PR #402: URL: https://github.com/apache/flume/pull/402 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@flume.apache.org

[GitHub] [flume] dependabot[bot] closed pull request #400: Bump jackson-databind from 2.13.2.1 to 2.13.4.1 in /flume-parent

2023-03-31 Thread via GitHub
dependabot[bot] closed pull request #400: Bump jackson-databind from 2.13.2.1 to 2.13.4.1 in /flume-parent URL: https://github.com/apache/flume/pull/400 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [flume] dependabot[bot] opened a new pull request, #404: Bump jackson-databind from 2.13.2.1 to 2.13.4.2 in /flume-parent

2023-03-31 Thread via GitHub
dependabot[bot] opened a new pull request, #404: URL: https://github.com/apache/flume/pull/404 Bumps [jackson-databind](https://github.com/FasterXML/jackson) from 2.13.2.1 to 2.13.4.2. Commits See full diff in https://github.com/FasterXML/jackson/commits;>compare view

[GitHub] [flume] dependabot[bot] commented on pull request #400: Bump jackson-databind from 2.13.2.1 to 2.13.4.1 in /flume-parent

2023-03-31 Thread via GitHub
dependabot[bot] commented on PR #400: URL: https://github.com/apache/flume/pull/400#issuecomment-1492070238 Superseded by #404. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [flume] nikita15p commented on pull request #402: [FLUME-3461] Fix vulnerability in netty-codec-haproxy

2023-03-30 Thread via GitHub
nikita15p commented on PR #402: URL: https://github.com/apache/flume/pull/402#issuecomment-1491334378 @rgoers can u please check this -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [flume-spring-boot] rgoers merged pull request #4: Fix some typos in RELEASE-NOTES.txt

2023-03-30 Thread via GitHub
rgoers merged PR #4: URL: https://github.com/apache/flume-spring-boot/pull/4 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [flume-search] asfgit merged pull request #4: Solve fastDateFormat unit test failed

2023-03-29 Thread via GitHub
asfgit merged PR #4: URL: https://github.com/apache/flume-search/pull/4 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [flume-search] xuwenping123 opened a new pull request, #4: Solve fastDateFormat unit test failed

2023-03-29 Thread via GitHub
xuwenping123 opened a new pull request, #4: URL: https://github.com/apache/flume-search/pull/4 Hi, as the title says, fix unit test case failure. ``` org.junit.ComparisonFailure: Expected :Coordinated Universal Time Actual :协调世界时间 at

[GitHub] [flume-search] rgoers closed pull request #1: Bump elasticsearch from 0.90.1 to 6.8.17

2023-03-28 Thread via GitHub
rgoers closed pull request #1: Bump elasticsearch from 0.90.1 to 6.8.17 URL: https://github.com/apache/flume-search/pull/1 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[GitHub] [flume-search] rgoers commented on pull request #1: Bump elasticsearch from 0.90.1 to 6.8.17

2023-03-28 Thread via GitHub
rgoers commented on PR #1: URL: https://github.com/apache/flume-search/pull/1#issuecomment-1487276019 Updating the dependency by itself will not be enough. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [flume-search] dependabot[bot] commented on pull request #1: Bump elasticsearch from 0.90.1 to 6.8.17

2023-03-28 Thread via GitHub
dependabot[bot] commented on PR #1: URL: https://github.com/apache/flume-search/pull/1#issuecomment-1487276099 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let

[GitHub] [flume-search] rgoers closed pull request #3: Flume search master branch failed to build

2023-03-28 Thread via GitHub
rgoers closed pull request #3: Flume search master branch failed to build URL: https://github.com/apache/flume-search/pull/3 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[GitHub] [flume-search] rgoers closed pull request #2: fix bug in ElasticSearchSink ,when use IndexnameBuilderContext can't get the right config

2023-03-28 Thread via GitHub
rgoers closed pull request #2: fix bug in ElasticSearchSink ,when use IndexnameBuilderContext can't get the right config URL: https://github.com/apache/flume-search/pull/2 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [flume-search] xuwenping123 opened a new pull request, #3: Flume search master branch failed to build

2023-03-28 Thread via GitHub
xuwenping123 opened a new pull request, #3: URL: https://github.com/apache/flume-search/pull/3 Please check the link for details, thanks. see https://issues.apache.org/jira/browse/FLUME-3464. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [flume] rgoers closed pull request #396: FLUME-3448 bump up to hadoop 3.2.2

2023-03-27 Thread via GitHub
rgoers closed pull request #396: FLUME-3448 bump up to hadoop 3.2.2 URL: https://github.com/apache/flume/pull/396 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[GitHub] [flume] rgoers commented on pull request #396: FLUME-3448 bump up to hadoop 3.2.2

2023-03-27 Thread via GitHub
rgoers commented on PR #396: URL: https://github.com/apache/flume/pull/396#issuecomment-1485838229 I pulled these changes, removed hbase-sink, fixed some checkstyle issues and then merged this. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [flume] nikita15p commented on pull request #396: FLUME-3448 bump up to hadoop 3.2.2

2023-03-27 Thread via GitHub
nikita15p commented on PR #396: URL: https://github.com/apache/flume/pull/396#issuecomment-1485160137 Sure @rgoers , if we are ok with removing hbase sink which I think is the best thing as most of the projects have moved out of hbase 1.x version including phoenix. I was looking into a

[GitHub] [flume] rgoers commented on pull request #396: FLUME-3448 bump up to hadoop 3.2.2

2023-03-27 Thread via GitHub
rgoers commented on PR #396: URL: https://github.com/apache/flume/pull/396#issuecomment-1484564061 What will you be doing? If it is just a matter of removing the HBase Sink I can handle that. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [flume] nikita15p commented on pull request #396: FLUME-3448 bump up to hadoop 3.2.2

2023-03-27 Thread via GitHub
nikita15p commented on PR #396: URL: https://github.com/apache/flume/pull/396#issuecomment-1484552069 @rgoers I am working on this and will update soon. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [flume] rgoers commented on pull request #396: FLUME-3448 bump up to hadoop 3.2.2

2023-03-26 Thread via GitHub
rgoers commented on PR #396: URL: https://github.com/apache/flume/pull/396#issuecomment-1484235323 I have spent several days looking into this and have come to the conclusion that there is no way to include the HBaseSink with these changes. The problem is that Guava seems to make

[GitHub] [flume] rgoers merged pull request #403: [FLUME-3462]the maven.restlet.org domain redirects to maven.restlet.talend.com

2023-03-25 Thread via GitHub
rgoers merged PR #403: URL: https://github.com/apache/flume/pull/403 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@flume.apache.org

[GitHub] [flume-twitter] rgoers commented on issue #1: Support for Twitter v2

2023-03-24 Thread via GitHub
rgoers commented on issue #1: URL: https://github.com/apache/flume-twitter/issues/1#issuecomment-1482359639 Pull requests are welcome! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [flume-twitter] ruwzeta opened a new issue, #1: Support for Twitter v2

2023-03-23 Thread via GitHub
ruwzeta opened a new issue, #1: URL: https://github.com/apache/flume-twitter/issues/1 Currently Twitter Source does not support streaming via the use of Twitter v2. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [flume] xuwenping123 commented on pull request #398: FLUME-3452. Upgrade libthrift version to fix CVE 2020-1938

2023-03-23 Thread via GitHub
xuwenping123 commented on PR #398: URL: https://github.com/apache/flume/pull/398#issuecomment-1481227708 > I apologize for making it seem like the unit test failures were due to this change. They were not. I just happened to notice that the CI builds were failing in the Thrift tests. When

[GitHub] [flume] xuwenping123 opened a new pull request, #403: [FLUME-3462]the maven.restlet.org domain redirects to maven.restlet.talend.com

2023-03-23 Thread via GitHub
xuwenping123 opened a new pull request, #403: URL: https://github.com/apache/flume/pull/403 As mentioned in the issue, to deal with the issue of certificate expiration due to the old domain name maven.restlet.org, visit the new domain name maven.restlet.talend.com. See:

[GitHub] [flume] nikita15p opened a new pull request, #402: [FLUME-3461] Fix vulnerability in netty-codec-haproxy

2023-03-22 Thread via GitHub
nikita15p opened a new pull request, #402: URL: https://github.com/apache/flume/pull/402 Fixing vulnerability in netty package version 4.1.72.Final Upgraded version to 4.1.86.Final -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [flume] rgoers commented on pull request #398: FLUME-3452. Upgrade libthrift version to fix CVE 2020-1938

2023-03-22 Thread via GitHub
rgoers commented on PR #398: URL: https://github.com/apache/flume/pull/398#issuecomment-1479776245 I apologize for making it seem like the unit test failures were due to this change. They were not. I just happened to notice that the CI builds were failing in the Thrift tests. When I used

[GitHub] [flume] xuwenping123 commented on pull request #398: FLUME-3452. Upgrade libthrift version to fix CVE 2020-1938

2023-03-22 Thread via GitHub
xuwenping123 commented on PR #398: URL: https://github.com/apache/flume/pull/398#issuecomment-1479660836 > The unit tests are failing when run with Java 11 as the truststore is using DSA. They work with Java 8. However, after upgrading from version 265 to 362 of Java 8 the tests fail. So

[GitHub] [flume] rgoers commented on pull request #398: FLUME-3452. Upgrade libthrift version to fix CVE 2020-1938

2023-03-21 Thread via GitHub
rgoers commented on PR #398: URL: https://github.com/apache/flume/pull/398#issuecomment-1478179238 The unit tests are failing when run with Java 11 as the truststore is using DSA. They work with Java 8. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [flume-spring-boot] rgoers merged pull request #2: Fix URL in .asf.yaml

2023-03-20 Thread via GitHub
rgoers merged PR #2: URL: https://github.com/apache/flume-spring-boot/pull/2 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [flume-spring-boot] rgoers commented on pull request #2: Fix URL in .asf.yaml

2023-03-20 Thread via GitHub
rgoers commented on PR #2: URL: https://github.com/apache/flume-spring-boot/pull/2#issuecomment-1476447817 Thanks for finding this one. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [flume-spring-boot] rgoers merged pull request #3: Update NOTICE.txt

2023-03-20 Thread via GitHub
rgoers merged PR #3: URL: https://github.com/apache/flume-spring-boot/pull/3 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [flume-spring-boot] rgoers commented on pull request #3: Update NOTICE.txt

2023-03-20 Thread via GitHub
rgoers commented on PR #3: URL: https://github.com/apache/flume-spring-boot/pull/3#issuecomment-1476445430 Well, OK. I don't think that is a big deal but I have nothing against it. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [flume-spring-boot] bessbd opened a new pull request, #3: Update NOTICE.txt

2023-03-20 Thread via GitHub
bessbd opened a new pull request, #3: URL: https://github.com/apache/flume-spring-boot/pull/3 This PR updates `NOTICE` so that it follows https://github.com/apache/flume/blob/trunk/NOTICE and https://github.com/apache/logging-log4j2/blob/2.x/NOTICE.txt -- This is an automated message

[GitHub] [flume] dependabot[bot] opened a new pull request, #401: Bump protobuf-java from 2.5.0 to 3.16.3 in /flume-parent

2023-03-19 Thread via GitHub
dependabot[bot] opened a new pull request, #401: URL: https://github.com/apache/flume/pull/401 Bumps [protobuf-java](https://github.com/protocolbuffers/protobuf) from 2.5.0 to 3.16.3. Release notes Sourced from https://github.com/protocolbuffers/protobuf/releases;>protobuf-java's

[GitHub] [flume] dependabot[bot] opened a new pull request, #400: Bump jackson-databind from 2.13.2.1 to 2.13.4.1 in /flume-parent

2023-03-19 Thread via GitHub
dependabot[bot] opened a new pull request, #400: URL: https://github.com/apache/flume/pull/400 Bumps [jackson-databind](https://github.com/FasterXML/jackson) from 2.13.2.1 to 2.13.4.1. Commits See full diff in https://github.com/FasterXML/jackson/commits;>compare view

[GitHub] [flume] dependabot[bot] closed pull request #378: Bump jetty-server from 9.4.41.v20210516 to 10.0.10

2023-03-19 Thread via GitHub
dependabot[bot] closed pull request #378: Bump jetty-server from 9.4.41.v20210516 to 10.0.10 URL: https://github.com/apache/flume/pull/378 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [flume] dependabot[bot] commented on pull request #378: Bump jetty-server from 9.4.41.v20210516 to 10.0.10

2023-03-19 Thread via GitHub
dependabot[bot] commented on PR #378: URL: https://github.com/apache/flume/pull/378#issuecomment-1475119570 Looks like org.eclipse.jetty:jetty-server is up-to-date now, so this is no longer needed. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [flume] dependabot[bot] opened a new pull request, #399: Bump hadoop-common from 2.10.1 to 3.2.4 in /flume-parent

2023-03-19 Thread via GitHub
dependabot[bot] opened a new pull request, #399: URL: https://github.com/apache/flume/pull/399 Bumps hadoop-common from 2.10.1 to 3.2.4. [![Dependabot compatibility

[GitHub] [flume] rgoers merged pull request #398: FLUME-3452. Upgrade libthrift version to fix CVE 2020-1938

2023-03-18 Thread via GitHub
rgoers merged PR #398: URL: https://github.com/apache/flume/pull/398 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@flume.apache.org

[GitHub] [flume] rgoers commented on pull request #398: FLUME-3452. Upgrade libthrift version to fix CVE 2020-1938

2023-03-18 Thread via GitHub
rgoers commented on PR #398: URL: https://github.com/apache/flume/pull/398#issuecomment-1474957786 Verified the build is still successful with this change. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [flume] xuwenping123 opened a new pull request, #398: FLUME-3452. Upgrade libthrift version to fix CVE 2020-1938

2023-03-17 Thread via GitHub
xuwenping123 opened a new pull request, #398: URL: https://github.com/apache/flume/pull/398 org.apache.thrift:libthrift:0.14.1 has dependency on tomcat-embed-core : 8.5.46 which is causing CVE 2020-1938. org.apache.thrift:libthrift:0.14.2 solved the problem for

[GitHub] [flume] rgoers merged pull request #392: fix a code typo

2023-03-17 Thread via GitHub
rgoers merged PR #392: URL: https://github.com/apache/flume/pull/392 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@flume.apache.org

[GitHub] [flume] rgoers commented on pull request #392: fix a code typo

2023-03-17 Thread via GitHub
rgoers commented on PR #392: URL: https://github.com/apache/flume/pull/392#issuecomment-1474276058 This looks ok to merge. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment.

[GitHub] [flume] rgoers merged pull request #393: FLUME-3445 - fix TestTwitterSource.testCarrotDateFormatBug on different locale env

2023-03-17 Thread via GitHub
rgoers merged PR #393: URL: https://github.com/apache/flume/pull/393 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@flume.apache.org

[GitHub] [flume] rgoers commented on pull request #393: FLUME-3445 - fix TestTwitterSource.testCarrotDateFormatBug on different locale env

2023-03-17 Thread via GitHub
rgoers commented on PR #393: URL: https://github.com/apache/flume/pull/393#issuecomment-1474275078 I see no problems with this. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [flume] rgoers merged pull request #397: [FLUME-3454] fixing exception while starting agent configured to jms source when passwordFile field is missing

2023-03-17 Thread via GitHub
rgoers merged PR #397: URL: https://github.com/apache/flume/pull/397 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@flume.apache.org

[GitHub] [flume] rgoers commented on pull request #397: [FLUME-3454] fixing exception while starting agent configured to jms source when passwordFile field is missing

2023-03-17 Thread via GitHub
rgoers commented on PR #397: URL: https://github.com/apache/flume/pull/397#issuecomment-1474273674 This looks innocent enough. I am just going to merge it. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [flume] nikita15p opened a new pull request, #397: [FLUME-3454] fixing exception while starting agent configured to jms source when passwordFile field is missing

2023-03-17 Thread via GitHub
nikita15p opened a new pull request, #397: URL: https://github.com/apache/flume/pull/397 Fixing exception while starting agent configured to jms source when passwordFile field is missing in the flume.conf file -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [flume-spring-boot] rgoers commented on issue #1: Spring Boot is not initializing Spring Cloud Config

2023-03-16 Thread via GitHub
rgoers commented on issue #1: URL: https://github.com/apache/flume-spring-boot/issues/1#issuecomment-1472749287 This issue was fixed in the initial commit to the repo - https://github.com/apache/flume-spring-boot/commit/1bd82e4cc5f844d2a95327be85448314d7e96e6f -- This is an automated

[GitHub] [flume-spring-boot] rgoers closed issue #1: Spring Boot is not initializing Spring Cloud Config

2023-03-16 Thread via GitHub
rgoers closed issue #1: Spring Boot is not initializing Spring Cloud Config URL: https://github.com/apache/flume-spring-boot/issues/1 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [flume-spring-boot] rgoers opened a new issue, #1: Spring Boot is not initializing Spring Cloud Config

2023-03-16 Thread via GitHub
rgoers opened a new issue, #1: URL: https://github.com/apache/flume-spring-boot/issues/1 It seems that having a Spring ApplicationContextInitializer that preforms a registry scan prevents Spring Boot from initializing Spring Cloud. A better approach is to simply require the application to

[GitHub] [flume] rgoers commented on pull request #396: FLUME-3448 bump up to hadoop 3.2.2

2023-03-12 Thread via GitHub
rgoers commented on PR #396: URL: https://github.com/apache/flume/pull/396#issuecomment-1465271233 I don't use Flume with Hadoop so I would not be the best qualified person to answer regarding hbase 1.x. I don't see any formal statement about HBase 1.x being EOL and, in fact, I see a 1.7.2

  1   2   >