[GitHub] [samza] jia-gao opened a new pull request, #1641: LISAMZA-27395 removing the current recursive call prevention logic

2022-11-08 Thread GitBox
jia-gao opened a new pull request, #1641: URL: https://github.com/apache/samza/pull/1641 Issue: We have observed logging data loss happen in StreamAppender The RC is that current implementation of the “[append()](https://github.com/apache/samza/blob/master/samza-log4j/src/main/java/or

[GitHub] [samza] mynameborat commented on pull request #1641: LISAMZA-27395 removing the current recursive call prevention logic

2022-11-08 Thread GitBox
mynameborat commented on PR #1641: URL: https://github.com/apache/samza/pull/1641#issuecomment-1307762089 Is the default support to prevent recursion not available when this change was introduced or did we upgrade to newer version of log4j that adds this prevention at the framework layer?

[GitHub] [samza] mynameborat commented on a diff in pull request #1640: SAMZA-2741: [Pipeline Drain] Fix ApplicationUtil.isHighLevelApiJob to work for anonymous and lambda `SamzaApplication` classes

2022-11-08 Thread GitBox
mynameborat commented on code in PR #1640: URL: https://github.com/apache/samza/pull/1640#discussion_r1017221725 ## samza-core/src/main/java/org/apache/samza/config/ApplicationConfig.java: ## @@ -46,6 +46,11 @@ public enum ApplicationMode { BATCH } + public enum Appli

[GitHub] [samza] xinyuiscool merged pull request #1640: SAMZA-2741: [Pipeline Drain] Fix ApplicationUtil.isHighLevelApiJob to work for anonymous and lambda `SamzaApplication` classes

2022-11-08 Thread GitBox
xinyuiscool merged PR #1640: URL: https://github.com/apache/samza/pull/1640 -- 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: commits-unsubscr...@samza.ap

[samza] branch master updated: SAMZA-2741: [Pipeline Drain] Fix ApplicationUtil.isHighLevelApiJob to work for anonymous SamzaApplication (#1640)

2022-11-08 Thread xinyu
This is an automated email from the ASF dual-hosted git repository. xinyu pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/samza.git The following commit(s) were added to refs/heads/master by this push: new 86d9b51cd SAMZA-2741: [Pipeline Drain] Fix Applic