Skip to content

[SPARK-21418][SQL] NoSuchElementException: None.get in DataSourceScanExec with sun.io.serialization.extendedDebugInfo=true - #19123

Closed
srowen wants to merge 1 commit into
apache:masterfrom
srowen:SPARK-21418
Closed

[SPARK-21418][SQL] NoSuchElementException: None.get in DataSourceScanExec with sun.io.serialization.extendedDebugInfo=true#19123
srowen wants to merge 1 commit into
apache:masterfrom
srowen:SPARK-21418

Conversation

@srowen

Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

If no SparkConf is available to Utils.redact, simply don't redact.

How was this patch tested?

Existing tests

@darabosdarabos left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks! That was super quick! 🚤

@SparkQA

Copy link
Copy Markdown

Test build #81388 has finished for PR 19123 at commit 735ca94.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@dongjoon-hyundongjoon-hyun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, LGTM, too.

@hvanhovell

Copy link
Copy Markdown
Contributor

LGTM, merging to master/2.2. Thanks for picking this up!

asfgit pushed a commit that referenced this pull request Sep 4, 2017
…Exec with sun.io.serialization.extendedDebugInfo=true
## What changes were proposed in this pull request?
If no SparkConf is available to Utils.redact, simply don't redact.
## How was this patch tested?
Existing tests
Author: Sean Owen <sowen@cloudera.com>
Closes#19123 from srowen/SPARK-21418.
(cherry picked from commit ca59445)
Signed-off-by: Herman van Hovell <hvanhovell@databricks.com>
@asfgitasfgit closed this in ca59445Sep 4, 2017
@srowen
srowen deleted the SPARK-21418 branch September 5, 2017 11:29
MatthewRBruce pushed a commit to Shopify/spark that referenced this pull request Jul 31, 2018
…Exec with sun.io.serialization.extendedDebugInfo=true
## What changes were proposed in this pull request?
If no SparkConf is available to Utils.redact, simply don't redact.
## How was this patch tested?
Existing tests
Author: Sean Owen <sowen@cloudera.com>
Closesapache#19123 from srowen/SPARK-21418.
(cherry picked from commit ca59445)
Signed-off-by: Herman van Hovell <hvanhovell@databricks.com>
Sign up for freeto 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.

5 participants

@srowen@SparkQA@hvanhovell@darabos@dongjoon-hyun