New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sources/Sinks can be launched using fat jars as well #3166
Conversation
run cpp tests |
@@ -120,39 +120,35 @@ void start() throws Exception { | |||
if (builtInSource != null) { | |||
sourceConfig.setArchive(builtInSource); | |||
} | |||
NarClassLoader classLoader; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does this mean we cannot run a built-in source or sink locally?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Built-ins should be passed to the localrun with a proper file location, so they can be
Motivation
Users should be able to just construct a fat jar for a Sources or a Sinks. While this capability existed in 2.2, subsequent cleanups removed the feature. This pr re-enables it.
Modifications
Describe the modifications you've done.
Result
After your change, what will change.