Skip to content

Resource pools api - #822

Closed
fireboy1919 wants to merge 43 commits into
apache:masterfrom
fireboy1919:resource_pools_api
Closed

Resource pools api#822
fireboy1919 wants to merge 43 commits into
apache:masterfrom
fireboy1919:resource_pools_api

Conversation

@fireboy1919

Copy link
Copy Markdown
Contributor

What is this PR for?

This is a step towards creating a REST endpoint from resource pools https://issues.apache.org/jira/browse/ZEPPELIN-699

It allows resources to be downloaded from results using a REST API

What type of PR is it?

Improvement

Todos

  • - Add ability to download resources via an API endpoint

What is the Jira issue?

Finishes: https://issues.apache.org/jira/browse/ZEPPELIN-699

Note, that this pull request is based upon these:
#782
#816

How should this be tested?

  1. Edit an interpreter
  2. Add a new property to the interpreter - the "ResourcePoolClass" - and assign it to "org.apache.zeppelin.resource.VFSResourcePool"
  3. Run the interpreter

You should see new resource files created in notebook/zeppelin_resources

Then go to
/api/notebook/results
to see the rest endpoint that lists all of the results.

View it with
/api/notebook/[noteid]/paragraph/[paragraph id]/results

Screenshots (if appropriate)

Questions:

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

@srinivas-startsmartlabs

Copy link
Copy Markdown

The api to view paragraph result is:

/api/notebook/[noteid]/paragraph/[paragraph id]/result

It was incorrectly mentioned as:

/api/notebook/[noteid]/paragraph/[paragraph id]/results

@felixcheung

Copy link
Copy Markdown
Member

How are we on this? @Leemoonsoo would you like to take a look?

@asfgitasfgit closed this in c38a0a0May 9, 2018
asfgit pushed a commit that referenced this pull request May 9, 2018
close#83close#86close#125close#133close#139close#146close#193close#203close#246close#262close#264close#273close#291close#299close#320close#347close#389close#413close#423close#543close#560close#658close#670close#728close#765close#777close#782close#783close#812close#822close#841close#843close#878close#884close#918close#989close#1076close#1135close#1187close#1231close#1304close#1316close#1361close#1385close#1390close#1414close#1422close#1425close#1447close#1458close#1466close#1485close#1492close#1495close#1497close#1536close#1545close#1561close#1577close#1600close#1603close#1678close#1695close#1739close#1748close#1765close#1767close#1776close#1783close#1799
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.

3 participants

@fireboy1919@srinivas-startsmartlabs@felixcheung