{"action":"created","issue":{"url":"https://api.github.com/repos/apache/incubator-zeppelin/issues/782","repository_url":"https://api.github.com/repos/apache/incubator-zeppelin","labels_url":"https://api.github.com/repos/apache/incubator-zeppelin/issues/782/labels{/name}","comments_url":"https://api.github.com/repos/apache/incubator-zeppelin/issues/782/comments","events_url":"https://api.github.com/repos/apache/incubator-zeppelin/issues/782/events","html_url":"https://github.com/apache/incubator-zeppelin/pull/782","id":141304548,"number":782,"title":"Enhanced resource pool serialization","user":{"login":"fireboy1919","id":8417951,"avatar_url":"https://avatars.githubusercontent.com/u/8417951?v=3","gravatar_id":"","url":"https://api.github.com/users/fireboy1919","html_url":"https://github.com/fireboy1919","followers_url":"https://api.github.com/users/fireboy1919/followers","following_url":"https://api.github.com/users/fireboy1919/following{/other_user}","gists_url":"https://api.github.com/users/fireboy1919/gists{/gist_id}","starred_url":"https://api.github.com/users/fireboy1919/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fireboy1919/subscriptions","organizations_url":"https://api.github.com/users/fireboy1919/orgs","repos_url":"https://api.github.com/users/fireboy1919/repos","events_url":"https://api.github.com/users/fireboy1919/events{/privacy}","received_events_url":"https://api.github.com/users/fireboy1919/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2016-03-16T15:12:34Z","updated_at":"2016-03-28T14:19:20Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/incubator-zeppelin/pulls/782","html_url":"https://github.com/apache/incubator-zeppelin/pull/782","diff_url":"https://github.com/apache/incubator-zeppelin/pull/782.diff","patch_url":"https://github.com/apache/incubator-zeppelin/pull/782.patch"},"body":"### What is this PR for?\r\nThis is a step towards creating a REST endpoint from resource pools  https://issues.apache.org/jira/browse/ZEPPELIN-699\r\n\r\nIt allows resources to be fully serializable so that the object that they contain can be passed around using thrift.\r\n\r\n### What type of PR is it?\r\nImprovement\r\n\r\n### Todos\r\n* [X] - Add serialization for rest endpoint\r\n\r\n### What is the Jira issue?\r\nPartially implements this:  https://issues.apache.org/jira/browse/ZEPPELIN-699\r\n\r\nNote, that this pull request is dependent upon this one:  \r\nhttps://github.com/apache/incubator-zeppelin/pull/781\r\n\r\nBefore that change is made, this will not compile.\r\n\r\n### How should this be tested?\r\nThere is a test class \"org.apache.zeppelin.ResourceSerializerTest\", but with no front-end, there isn't an easy manual way to test this.\r\n\r\nHowever, these steps will work:\r\n\r\n1. Start Zeppelin\r\n2. Run a paragraph.\r\n3.  Inspect \"org.apache.zeppelin.resource.ResourcePoolUtils.getAllResources()\".  You will see that this paragraph's results are included in this resource pool.\r\n\r\n### Screenshots (if appropriate)\r\n\r\n### Questions:\r\n* Does the licenses files need update? No\r\n* Is there breaking changes for older versions? No\r\n* Does this needs documentation?  No.\r\n"},"comment":{"url":"https://api.github.com/repos/apache/incubator-zeppelin/issues/comments/202411857","html_url":"https://github.com/apache/incubator-zeppelin/pull/782#issuecomment-202411857","issue_url":"https://api.github.com/repos/apache/incubator-zeppelin/issues/782","id":202411857,"user":{"login":"fireboy1919","id":8417951,"avatar_url":"https://avatars.githubusercontent.com/u/8417951?v=3","gravatar_id":"","url":"https://api.github.com/users/fireboy1919","html_url":"https://github.com/fireboy1919","followers_url":"https://api.github.com/users/fireboy1919/followers","following_url":"https://api.github.com/users/fireboy1919/following{/other_user}","gists_url":"https://api.github.com/users/fireboy1919/gists{/gist_id}","starred_url":"https://api.github.com/users/fireboy1919/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fireboy1919/subscriptions","organizations_url":"https://api.github.com/users/fireboy1919/orgs","repos_url":"https://api.github.com/users/fireboy1919/repos","events_url":"https://api.github.com/users/fireboy1919/events{/privacy}","received_events_url":"https://api.github.com/users/fireboy1919/received_events","type":"User","site_admin":false},"created_at":"2016-03-28T14:19:20Z","updated_at":"2016-03-28T14:19:20Z","body":"That makes sense.  The issue is that \"ResourcePoolUtils\" currently doesn't use that enhanced method, and if you're trying to get a specific resource, that looks like the proper path to do it.\r\n\r\nHow about I create a pull request that modified \"ResourcePoolUtils.getAllResourcesExcept\" so that the returned resources from the remote interpreter process are actually remote resources?"}}