Skip to content

Conversation

@minahlee
Copy link
Member

@minahlee minahlee commented Jul 5, 2016

What is this PR for?

Catch Exception in case dialog is removed from dom to fix flaky test of ZeppelinIT. testSparkInterpreterDependencyLoading

What type of PR is it?

Hot Fix

What is the Jira issue?

ZEPPELIN-1110

Questions:

  • Does the licenses files need update? no
  • Is there breaking changes for older versions? no
  • Does this needs documentation? no

@minahlee minahlee changed the title [ZEPPELIN-1110] Catch Exception in ZeppelinIT. testSparkInterpreterDependencyLoading in case dialog is removed from dom [ZEPPELIN-1110] Catch Exception in ZeppelinIT. testSparkInterpreterDependencyLoading Jul 5, 2016
@bzz
Copy link
Member

bzz commented Jul 5, 2016

Looks good to me

@prabhjyotsingh
Copy link
Contributor

👍

@minahlee
Copy link
Member Author

minahlee commented Jul 5, 2016

Let me merge this into master as hotfix once the selenium test of last commit becomes green.

@bzz
Copy link
Member

bzz commented Jul 5, 2016

CI is green now! :shipit:

@asfgit asfgit closed this in 867facb Jul 5, 2016
PhilippGrulich pushed a commit to SWC-SENSE/zeppelin that referenced this pull request Aug 8, 2016
…pendencyLoading

### What is this PR for?
Catch Exception in case dialog is removed from dom to fix flaky test of `ZeppelinIT. testSparkInterpreterDependencyLoading`

### What type of PR is it?
Hot Fix

### What is the Jira issue?
[ZEPPELIN-1110](https://issues.apache.org/jira/browse/ZEPPELIN-1110)

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no

Author: Mina Lee <[email protected]>

Closes apache#1129 from minahlee/ZEPPELIN-1110 and squashes the following commits:

c681410 [Mina Lee] Retrigger CI
69da3c1 [Mina Lee] Retrigger CI
21435be [Mina Lee] Retrigger CI
8cd7065 [Mina Lee] catch StaleElementReferenceException in case dialog is removed from dom
@minahlee minahlee deleted the ZEPPELIN-1110 branch November 2, 2016 05:01
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