yashmayya opened a new pull request, #14082: URL: https://github.com/apache/pinot/pull/14082
- This is a minor cleanup PR that fixes the incorrect log lines in the `PluginManager` class. - The `PluginManager` class supports configuration via two Java system properties `plugins.dir` and `plugins.include`. - [System Properties in Java](https://docs.oracle.com/javase/tutorial/essential/environment/sysprop.html) are set via JVM arguments at startup (like `-Dplugins.dir`) and are distinct from OS level environment variables. - Alternatively, we could also trivially add support for reading these two configurations from equivalent environment variables as well (but it's generally better to use system properties since it's scoped to the JVM process vs OS-wide environment variables). -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
