Skip to content
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

DSP-19969 allow spark executor log to separate file #30

Open
wants to merge 1 commit into
base: dse-2
Choose a base branch
from

Conversation

alexliu68
Copy link

No description provided.

@RussellSpitzer
Copy link

I'm not sure I understand this, the Jira is about a directory which is being created, but this pr is to add a command which changes the local executor Directory for logging?

Is it that the default "LoggingOptions" env variable is creating the wrong directory? Shouldn't we just change it then?

@RussellSpitzer
Copy link

Looking at the original

# Override logging options to provide per-context logging
LOGGING_OPTS="$LOGGING_OPTS_FILE
              -DLOG_DIR=$1"

Log dir should be set to the first arg, checking our dse start script we just pass through args

                if [ "$DSERM" == "v1" ]; then
                    SJS_LAUNCHER="$SPARK_HOME/bin/spark-submit" run_with_framework "$SPARK_JOBSERVER_HOME"/manager_start.sh "$@"
                else
                    SJS_LAUNCHER="$DSE_COMPONENTS_ROOT/spark-launcher/bin/spark-submit" "$SPARK_JOBSERVER_HOME"/manager_start.sh "$@"
                fi

@alexliu68
Copy link
Author

Spark job server by default sets executor logging to spark-job-server.log which is spark job server (app driver) log. This ticket removes spark job default logging to executor, and use default spark executor logging. It keeps spark job server to spark-job-server.log. We can't just change LoggingOptions for both, we have to keep LoggingOptions for spark job server and logging executor separately

@alexliu68
Copy link
Author

@RussellSpitzer I replied your comment

@RussellSpitzer
Copy link

So now the default behavior is logging to SPARK_WORK/app/executor?
And what does -s do?

@alexliu68
Copy link
Author

default doesn't change. -s option changes the executor log to separate spark log instead of default spark job server.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants