Skip to content

ZEPPELIN-458 Fix testAbortParagraphStatusOnInterpreterRestart - #468

Closed
jongyoul wants to merge 1 commit into
apache:masterfrom
jongyoul:ZEPPELIN-458
Closed

ZEPPELIN-458 Fix testAbortParagraphStatusOnInterpreterRestart#468
jongyoul wants to merge 1 commit into
apache:masterfrom
jongyoul:ZEPPELIN-458

Conversation

@jongyoul

Copy link
Copy Markdown
Member

- Changed how to check some paragraph is aborted
@jongyoul

Copy link
Copy Markdown
MemberAuthor

I've changed restart timing and checked the status of paragraphs. The test passed if at least one of paragraphs has Status.ABORT.

@Leemoonsoo

Copy link
Copy Markdown
Member

LGTM

@jongyoul

Copy link
Copy Markdown
MemberAuthor

Thanks.

@bzz

bzz commented Nov 26, 2015

Copy link
Copy Markdown
Member

Looks good to me

@jongyoul

Copy link
Copy Markdown
MemberAuthor

Ready to merge

@jongyoul

Copy link
Copy Markdown
MemberAuthor

Merging if there isn't another discussion.

@khalidhuseynov

Copy link
Copy Markdown
Member

Thanks for the fix, LGTM!

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.

4 participants

@jongyoul@Leemoonsoo@bzz@khalidhuseynov