Skip to content

Conversation

@vanzin
Copy link
Contributor

@vanzin vanzin commented Mar 8, 2018

This change restores functionality that was inadvertently removed as part
of the fix for SPARK-22372.

Also modified an existing unit test to make sure the feature works as intended.

…ffect.

This change restores functionality that was inadvertently removed as part
of the fix for SPARK-22372.

Also modified an existing unit test to make sure the feature works as intended.
@SparkQA
Copy link

SparkQA commented Mar 9, 2018

Test build #88104 has finished for PR 20776 at commit 5abac2c.

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

@jerryshao
Copy link
Contributor

LGTM.

@vanzin
Copy link
Contributor Author

vanzin commented Mar 9, 2018

Merging to master / 2.3.

asfgit pushed a commit that referenced this pull request Mar 9, 2018
…ffect.

This change restores functionality that was inadvertently removed as part
of the fix for SPARK-22372.

Also modified an existing unit test to make sure the feature works as intended.

Author: Marcelo Vanzin <[email protected]>

Closes #20776 from vanzin/SPARK-23630.

(cherry picked from commit 2c36736)
Signed-off-by: Marcelo Vanzin <[email protected]>
@asfgit asfgit closed this in 2c36736 Mar 9, 2018
@vanzin vanzin deleted the SPARK-23630 branch March 12, 2018 20:27
peter-toth pushed a commit to peter-toth/spark that referenced this pull request Oct 6, 2018
…ffect.

This change restores functionality that was inadvertently removed as part
of the fix for SPARK-22372.

Also modified an existing unit test to make sure the feature works as intended.

Author: Marcelo Vanzin <[email protected]>

Closes apache#20776 from vanzin/SPARK-23630.

(cherry picked from commit 2c36736)
Signed-off-by: Marcelo Vanzin <[email protected]>
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.

4 participants