{"action":"created","issue":{"url":"https://api.github.com/repos/caskdata/hydrator-plugins/issues/235","repository_url":"https://api.github.com/repos/caskdata/hydrator-plugins","labels_url":"https://api.github.com/repos/caskdata/hydrator-plugins/issues/235/labels{/name}","comments_url":"https://api.github.com/repos/caskdata/hydrator-plugins/issues/235/comments","events_url":"https://api.github.com/repos/caskdata/hydrator-plugins/issues/235/events","html_url":"https://github.com/caskdata/hydrator-plugins/pull/235","id":149909792,"number":235,"title":"CDAP-5664 Use external dataset naming feature in hydrator-plugins for lineage/metadata etc","user":{"login":"gokulavasan","id":558459,"avatar_url":"https://avatars.githubusercontent.com/u/558459?v=3","gravatar_id":"","url":"https://api.github.com/users/gokulavasan","html_url":"https://github.com/gokulavasan","followers_url":"https://api.github.com/users/gokulavasan/followers","following_url":"https://api.github.com/users/gokulavasan/following{/other_user}","gists_url":"https://api.github.com/users/gokulavasan/gists{/gist_id}","starred_url":"https://api.github.com/users/gokulavasan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gokulavasan/subscriptions","organizations_url":"https://api.github.com/users/gokulavasan/orgs","repos_url":"https://api.github.com/users/gokulavasan/repos","events_url":"https://api.github.com/users/gokulavasan/events{/privacy}","received_events_url":"https://api.github.com/users/gokulavasan/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"poornachandra","id":3181212,"avatar_url":"https://avatars.githubusercontent.com/u/3181212?v=3","gravatar_id":"","url":"https://api.github.com/users/poornachandra","html_url":"https://github.com/poornachandra","followers_url":"https://api.github.com/users/poornachandra/followers","following_url":"https://api.github.com/users/poornachandra/following{/other_user}","gists_url":"https://api.github.com/users/poornachandra/gists{/gist_id}","starred_url":"https://api.github.com/users/poornachandra/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/poornachandra/subscriptions","organizations_url":"https://api.github.com/users/poornachandra/orgs","repos_url":"https://api.github.com/users/poornachandra/repos","events_url":"https://api.github.com/users/poornachandra/events{/privacy}","received_events_url":"https://api.github.com/users/poornachandra/received_events","type":"User","site_admin":false},"milestone":null,"comments":5,"created_at":"2016-04-20T23:01:37Z","updated_at":"2016-04-21T19:50:42Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/caskdata/hydrator-plugins/pulls/235","html_url":"https://github.com/caskdata/hydrator-plugins/pull/235","diff_url":"https://github.com/caskdata/hydrator-plugins/pull/235.diff","patch_url":"https://github.com/caskdata/hydrator-plugins/pull/235.patch"},"body":"TODO:\r\n\r\ni) Fix unit-tests since the new property is a non-nullable property\r\nii) Update documentation once the description is agreed upon\r\niii) Modify all external sources/sinks (non-CDAP dataset) to have referenceName property"},"comment":{"url":"https://api.github.com/repos/caskdata/hydrator-plugins/issues/comments/213087628","html_url":"https://github.com/caskdata/hydrator-plugins/pull/235#issuecomment-213087628","issue_url":"https://api.github.com/repos/caskdata/hydrator-plugins/issues/235","id":213087628,"user":{"login":"gokulavasan","id":558459,"avatar_url":"https://avatars.githubusercontent.com/u/558459?v=3","gravatar_id":"","url":"https://api.github.com/users/gokulavasan","html_url":"https://github.com/gokulavasan","followers_url":"https://api.github.com/users/gokulavasan/followers","following_url":"https://api.github.com/users/gokulavasan/following{/other_user}","gists_url":"https://api.github.com/users/gokulavasan/gists{/gist_id}","starred_url":"https://api.github.com/users/gokulavasan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gokulavasan/subscriptions","organizations_url":"https://api.github.com/users/gokulavasan/orgs","repos_url":"https://api.github.com/users/gokulavasan/repos","events_url":"https://api.github.com/users/gokulavasan/events{/privacy}","received_events_url":"https://api.github.com/users/gokulavasan/received_events","type":"User","site_admin":false},"created_at":"2016-04-21T19:50:42Z","updated_at":"2016-04-21T19:50:42Z","body":"@albertshau @poornachandra The deprecated Id class in cdap-proto has a 'isValidId(String)' method. I can't find an equivalent method in the new EntityId class. If we add one, then I think we can depend on cdap-proto and check if referenceName is a validId during the configurePipeline step and thus fail the creation of the pipeline itself, instead of failing during the MapReduce runtime."}}