[PR] [Flink 32701] [cep] Fix CEP Operator Memory Leak Issue [flink]

2024-01-14 Thread via GitHub
pdream17 opened a new pull request, #24084: URL: https://github.com/apache/flink/pull/24084 ## What is the purpose of the change Resolve memory leak issue of CEP operator caused by NFA State and SharedBuffer.eventsCount. Details have been added to [32701](https://issues.apach

Re: [PR] [Flink 32701] [cep] Fix CEP Operator Memory Leak Issue [flink]

2024-01-14 Thread via GitHub
pdream17 commented on code in PR #24084: URL: https://github.com/apache/flink/pull/24084#discussion_r1451724830 ## flink-libraries/flink-cep/src/test/java/org/apache/flink/cep/nfa/NFAStateAccessTest.java: ## @@ -81,8 +81,8 @@ public void testComplexBranchingAfterZeroOrMore() thr

Re: [PR] [Flink 32701] [cep] Fix CEP Operator Memory Leak Issue [flink]

2024-01-14 Thread via GitHub
flinkbot commented on PR #24084: URL: https://github.com/apache/flink/pull/24084#issuecomment-1890940889 ## CI report: * d7cb74e7229af8076e7e69b12d7c94842822f7b8 UNKNOWN Bot commands The @flinkbot bot supports the following commands: - `@flinkbot run azure`

Re: [PR] [Flink 32701] [cep] Fix CEP Operator Memory Leak Issue [flink]

2024-01-17 Thread via GitHub
MartijnVisser commented on PR #24084: URL: https://github.com/apache/flink/pull/24084#issuecomment-1895922772 @pdream17 Thanks for the PR, but you'll need to apply spotless since currently it already fails on the compilation step of the CI -- This is an automated message from the Apache G

Re: [PR] [Flink 32701] [cep] Fix CEP Operator Memory Leak Issue [flink]

2024-01-19 Thread via GitHub
pdream17 commented on PR #24084: URL: https://github.com/apache/flink/pull/24084#issuecomment-1900312312 Thanks @MartijnVisser . Have corrected it and CI phase is success. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

Re: [PR] [Flink 32701] [cep] Fix CEP Operator Memory Leak Issue [flink]

2024-01-19 Thread via GitHub
MartijnVisser commented on PR #24084: URL: https://github.com/apache/flink/pull/24084#issuecomment-1900334701 @pdream17 You'll need to rebase, and not use merge-commits since PRs that contain merge commits can't be merged -- This is an automated message from the Apache Git Service. To res

Re: [PR] [Flink 32701] [cep] Fix CEP Operator Memory Leak Issue [flink]

2024-01-20 Thread via GitHub
pdream17 commented on PR #24084: URL: https://github.com/apache/flink/pull/24084#issuecomment-1902191249 @MartijnVisser Its done and CI phase is success as well -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

Re: [PR] [Flink 32701] [cep] Fix CEP Operator Memory Leak Issue [flink]

2024-03-21 Thread via GitHub
dawidwys merged PR #24084: URL: https://github.com/apache/flink/pull/24084 -- 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: issues-unsubscr...@flink.apac