lhotari opened a new pull request #9673:
URL: https://github.com/apache/pulsar/pull/9673
### Motivation
`org.apache.pulsar.functions.LocalRunner` currently has several issues:
* cleanup and shutdown isn't handled in tests
* it doesn't support .nar files that aren't on the classpath
* adding support for .nar files that aren't on the classpath will help in
the PIP-62 migration so that an integration test can run tests with a .nar file
that has been built in some other build (for example)
### Modifications
- Make LocalRunner work properly with .nar files that aren't on the
classpath
- also support the previous choice of providing the implementation classes
on the Thread context classloader classpath
- Pass nar file locations as system properties to tests:
- Pass pulsar-io cassandra and twitter nar file locations as system
properties
to tests
- Pass pulsar-io-data-generator.nar and pulsar-functions-api-examples.jar
file locations as system properties to tests
- Improve LocalRunner cleanup/shutdown which wasn't handled before.
- Fix invalid test in PackagesApiTest
- Make PulsarFunction tests and Sink/Source tests work with .nar files which
aren't on the classpath.
- Extract FileServer in tests to simplify the test code
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
