xintongsong commented on PR #514: URL: https://github.com/apache/flink-agents/pull/514#issuecomment-3844930564
@klaudworks, thank you for reporting and fixing this bug. This is indeed a valid problem, and the fix sounds good. @wenjin272, please help check the details? The 0.2.0 version is already in RC vote, and will soon be releases. Given that there's a workaround to this issue (placing user-defined resource in `flink/lib`), I'd suggest not to withdraw the RC and fix this in the next patch release. WDYT? Additionally, I think we should: - Create an issue describing the problem, workaround and track the fix version. This is helpful for other people who run into the same problem, because people usually tend to search in issues rather than PRs when they run into problems. @klaudworks, would you like to do that? - We are planning to add some e2e tests that actually submit jobs into a flink standalone cluster, instead of currently running in a mini-cluster. I think user-defined resource should be included as one of the test cases. @wenjin272 -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
