Skip to content

Fix restapi testcase#572

Closed
astroshim wants to merge 5 commits intoapache:masterfrom
astroshim:fix-restapi-testcase
Closed

Fix restapi testcase#572
astroshim wants to merge 5 commits intoapache:masterfrom
astroshim:fix-restapi-testcase

Conversation

@astroshim
Copy link
Copy Markdown
Contributor

@astroshim astroshim commented Dec 27, 2015

What is this PR for?

To fix REST API testcase.

What type of PR is it?

Bug Fix

Is there a relevant Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-536

How should this be tested?

Run testcase.

Questions:

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

@HeartSaVioR
Copy link
Copy Markdown
Contributor

HeartSaVioR commented Dec 28, 2015

Looks good to me. Though build failure is not related to your modification, could you trigger Travis build by close - reopen?

@astroshim
Copy link
Copy Markdown
Contributor Author

astroshim commented Dec 28, 2015

@HeartSaVioR Thank you for your reply.

@astroshim astroshim closed this Dec 28, 2015
@astroshim astroshim reopened this Dec 28, 2015
@astroshim
Copy link
Copy Markdown
Contributor Author

astroshim commented Dec 29, 2015

The build error is not related to this issue. But it occurred again!

...
15/12/28 04:23:52 WARN TaskSchedulerImpl: Initial job has not accepted any resources; check your cluster UI to ensure that workers are registered and have sufficient resources
15/12/28 04:24:07 WARN TaskSchedulerImpl: Initial job has not accepted any resources; check your cluster UI to ensure that workers are registered and have sufficient resources
15/12/28 04:24:22 WARN TaskSchedulerImpl: Initial job has not accepted any resources; check your cluster UI to ensure that workers are registered and have sufficient resources


Your test run exceeded 120 minutes. 

One possible solution is to split up your test run.

@Leemoonsoo
Copy link
Copy Markdown
Member

Leemoonsoo commented Jan 3, 2016

LGTM and merge if there're no more discussions

@asfgit asfgit closed this in fb75cb3 Jan 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@astroshim @HeartSaVioR @Leemoonsoo