piotrp commented on PR #27697: URL: https://github.com/apache/flink/pull/27697#issuecomment-3996918731
Looks ok to me. Just a small nit, but I'm not sure how relevant - originally I put `TestClass` in `org.apache.notflink` namespace to demonstrate that special treatment of `org.apache.flink` package was one of the reasons that class loading didn't work correctly. Personally I believe that due to this special treatment of `org.apache.flink` at least some user classes in tests should reside in namespaces that don't get this special treatment, and this should be fine so long as these don't leak into any published artifacts. -- 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]
