Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/15009#discussion_r82075274
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/SparkLauncher.java ---
@@ -488,11 +546,22 @@ public Process launch() throws IOException {
* In all cases, the logger name will start with
"org.apache.spark.launcher.app", to fit more
* easily into the configuration of commonly-used logging systems.
*
+ * If the applications are launched as a thread, the {@link
SparkLauncher#redirectError} and
+ * {@link SparkLauncher#redirectToLog}, are not supported at this time.
The existing process
+ * stdout and stderr will get all the log entries.
+ *
* @since 1.6.0
* @param listeners Listeners to add to the handle before the app is
launched.
* @return A handle for the launched application.
*/
public SparkAppHandle startApplication(SparkAppHandle.Listener...
listeners) throws IOException {
+ if(launchSparkSubmitAsThread) {
--- End diff --
add space after if
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
