[GitHub] [flink] pgaref commented on pull request #22643: [FLINK-31894][runtime] ExceptionHistory and REST API failure label integration

2023-06-01 Thread via GitHub
pgaref commented on PR #22643: URL: https://github.com/apache/flink/pull/22643#issuecomment-157247 >  good job; let's just get rid of the separate FailureLabel class as discussed offline Thanks much @dmvk ! Updated :) -- This is an automated message from the Apache Git

[GitHub] [flink] pgaref commented on pull request #22643: [FLINK-31894][runtime] ExceptionHistory and REST API failure label integration

2023-05-31 Thread via GitHub
pgaref commented on PR #22643: URL: https://github.com/apache/flink/pull/22643#issuecomment-1571174198 > The biggest question I have is whether the filters are supposed to be ANDed or ORed 樂 Right now if a single filter matches we keep the exception. Changed this to ANDed filter