Skip to content

Conversation

@zsxwing
Copy link
Member

@zsxwing zsxwing commented Nov 17, 2016

What changes were proposed in this pull request?

HDFS write may just hang until timeout if some network error happens. It's better to enable interrupts to allow stopping the query fast on HDFS.

This PR just changes the logic to only disable interrupts for local file system, as HADOOP-10622 only happens for local file system.

How was this patch tested?

Jenkins

@zsxwing
Copy link
Member Author

zsxwing commented Nov 17, 2016

cc @tdas

@SparkQA
Copy link

SparkQA commented Nov 17, 2016

Test build #68738 has finished for PR 15911 at commit 02cc010.

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

@tdas
Copy link
Contributor

tdas commented Nov 18, 2016

LGTM. Merging to master, 2.1, and 2.0

asfgit pushed a commit that referenced this pull request Nov 19, 2016
## What changes were proposed in this pull request?

HDFS `write` may just hang until timeout if some network error happens. It's better to enable interrupts to allow stopping the query fast on HDFS.

This PR just changes the logic to only disable interrupts for local file system, as HADOOP-10622 only happens for local file system.

## How was this patch tested?

Jenkins

Author: Shixiong Zhu <[email protected]>

Closes #15911 from zsxwing/interrupt-on-dfs.

(cherry picked from commit e5f5c29)
Signed-off-by: Tathagata Das <[email protected]>
@asfgit asfgit closed this in e5f5c29 Nov 19, 2016
asfgit pushed a commit that referenced this pull request Nov 19, 2016
## What changes were proposed in this pull request?

HDFS `write` may just hang until timeout if some network error happens. It's better to enable interrupts to allow stopping the query fast on HDFS.

This PR just changes the logic to only disable interrupts for local file system, as HADOOP-10622 only happens for local file system.

## How was this patch tested?

Jenkins

Author: Shixiong Zhu <[email protected]>

Closes #15911 from zsxwing/interrupt-on-dfs.

(cherry picked from commit e5f5c29)
Signed-off-by: Tathagata Das <[email protected]>
@zsxwing zsxwing deleted the interrupt-on-dfs branch November 19, 2016 00:18
uzadude pushed a commit to uzadude/spark that referenced this pull request Jan 27, 2017
## What changes were proposed in this pull request?

HDFS `write` may just hang until timeout if some network error happens. It's better to enable interrupts to allow stopping the query fast on HDFS.

This PR just changes the logic to only disable interrupts for local file system, as HADOOP-10622 only happens for local file system.

## How was this patch tested?

Jenkins

Author: Shixiong Zhu <[email protected]>

Closes apache#15911 from zsxwing/interrupt-on-dfs.
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