{"action":"created","issue":{"url":"https://api.github.com/repos/apache/zeppelin/issues/1647","repository_url":"https://api.github.com/repos/apache/zeppelin","labels_url":"https://api.github.com/repos/apache/zeppelin/issues/1647/labels{/name}","comments_url":"https://api.github.com/repos/apache/zeppelin/issues/1647/comments","events_url":"https://api.github.com/repos/apache/zeppelin/issues/1647/events","html_url":"https://github.com/apache/zeppelin/pull/1647","id":189776329,"number":1647,"title":"[ZEPPELIN-1013] Don't run paragraph on selectbox change","user":{"login":"Mleekko","id":1646177,"avatar_url":"https://avatars.githubusercontent.com/u/1646177?v=3","gravatar_id":"","url":"https://api.github.com/users/Mleekko","html_url":"https://github.com/Mleekko","followers_url":"https://api.github.com/users/Mleekko/followers","following_url":"https://api.github.com/users/Mleekko/following{/other_user}","gists_url":"https://api.github.com/users/Mleekko/gists{/gist_id}","starred_url":"https://api.github.com/users/Mleekko/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Mleekko/subscriptions","organizations_url":"https://api.github.com/users/Mleekko/orgs","repos_url":"https://api.github.com/users/Mleekko/repos","events_url":"https://api.github.com/users/Mleekko/events{/privacy}","received_events_url":"https://api.github.com/users/Mleekko/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-11-16T17:15:44Z","updated_at":"2016-11-23T12:33:42Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/zeppelin/pulls/1647","html_url":"https://github.com/apache/zeppelin/pull/1647","diff_url":"https://github.com/apache/zeppelin/pull/1647.diff","patch_url":"https://github.com/apache/zeppelin/pull/1647.patch"},"body":"### What is this PR for?\r\nEnabling to change values of multiple selectboxes witout running the paragraph\r\n\r\n### What type of PR is it?\r\n[Bug Fix]\r\n\r\n### Todos\r\nNo\r\n\r\n### What is the Jira issue?\r\n[ZEPPELIN-1013]\r\n\r\n### How should this be tested?\r\n1. Create a paragraph with  a selectbox via z.select(...)\r\n2. Select any value other than currently selected\r\nActual result:   value changes and paragraph gets executed\r\nExpected result: value changes\r\n\r\n### Screenshots (if appropriate)\r\n\r\n### Questions:\r\n* Does the licenses files need update?\r\nNo\r\n* Is there breaking changes for older versions?\r\nNo\r\n* Does this needs documentation?\r\nNo"},"comment":{"url":"https://api.github.com/repos/apache/zeppelin/issues/comments/262501586","html_url":"https://github.com/apache/zeppelin/pull/1647#issuecomment-262501586","issue_url":"https://api.github.com/repos/apache/zeppelin/issues/1647","id":262501586,"user":{"login":"1ambda","id":4968473,"avatar_url":"https://avatars.githubusercontent.com/u/4968473?v=3","gravatar_id":"","url":"https://api.github.com/users/1ambda","html_url":"https://github.com/1ambda","followers_url":"https://api.github.com/users/1ambda/followers","following_url":"https://api.github.com/users/1ambda/following{/other_user}","gists_url":"https://api.github.com/users/1ambda/gists{/gist_id}","starred_url":"https://api.github.com/users/1ambda/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/1ambda/subscriptions","organizations_url":"https://api.github.com/users/1ambda/orgs","repos_url":"https://api.github.com/users/1ambda/repos","events_url":"https://api.github.com/users/1ambda/events{/privacy}","received_events_url":"https://api.github.com/users/1ambda/received_events","type":"User","site_admin":false},"created_at":"2016-11-23T12:33:42Z","updated_at":"2016-11-23T12:33:42Z","body":"## Review Summary \r\n\r\nIt works as @Mleekko described (I attached GIF)\r\n\r\nI think this is better for the default behavior of dynamic selectbox. Since user can choose to run or not to run. Users do not have an option in current implementation. ([ZEPPELIN-1013](https://issues.apache.org/jira/browse/ZEPPELIN-1013) is a complain about that)\r\n\r\n@Mleekko \r\n\r\n- We can enforce this implementation by adding a test? So that someone can't change the default behavior unintentionally.\r\n\r\n### Screenshot (before)\r\n![before_1647](https://cloud.githubusercontent.com/assets/4968473/20561901/36b8e57c-b1c4-11e6-90f7-c5c4379d7b9c.gif)\r\n\r\n### Screenshot (after)\r\n![after_1647](https://cloud.githubusercontent.com/assets/4968473/20561903/3c5605fa-b1c4-11e6-8057-bed0ed006084.gif)\r\n\r\n\r\n"}}
{"actor":{"display_login":"1ambda"}}