{"action":"created","issue":{"url":"https://api.github.com/repos/scalingdata/cdk/issues/13","labels_url":"https://api.github.com/repos/scalingdata/cdk/issues/13/labels{/name}","comments_url":"https://api.github.com/repos/scalingdata/cdk/issues/13/comments","events_url":"https://api.github.com/repos/scalingdata/cdk/issues/13/events","html_url":"https://github.com/scalingdata/cdk/pull/13","id":68794085,"number":13,"title":"CDK-615: Permissions issue writing to a partitioned Hive external dataset","user":{"login":"joey","id":249392,"avatar_url":"https://avatars.githubusercontent.com/u/249392?v=3","gravatar_id":"","url":"https://api.github.com/users/joey","html_url":"https://github.com/joey","followers_url":"https://api.github.com/users/joey/followers","following_url":"https://api.github.com/users/joey/following{/other_user}","gists_url":"https://api.github.com/users/joey/gists{/gist_id}","starred_url":"https://api.github.com/users/joey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/joey/subscriptions","organizations_url":"https://api.github.com/users/joey/orgs","repos_url":"https://api.github.com/users/joey/repos","events_url":"https://api.github.com/users/joey/events{/privacy}","received_events_url":"https://api.github.com/users/joey/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-04-15T21:46:50Z","updated_at":"2015-04-15T23:11:58Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/scalingdata/cdk/pulls/13","html_url":"https://github.com/scalingdata/cdk/pull/13","diff_url":"https://github.com/scalingdata/cdk/pull/13.diff","patch_url":"https://github.com/scalingdata/cdk/pull/13.patch"},"body":"* This patch reverses course from the previous one. Depending on the\r\n  dataset implementation, you may or may not want Kite creating the\r\n  directories.\r\n* This patch moves the partitionAdded callback to happen before new\r\n  partition directories are created to give the listener a chance\r\n  to create them in Kite or externally.\r\n* In particular, external Hive datasets will create the directories on\r\n  the Kite side while managed Hive datasets will let the Hive metastore\r\n  create them.\r\n* This doesn't affect file system datasets as they don't have listeners\r\n  regisitered unless they're manulaly created with a custom\r\n  MetadataProvider\r\n* Added a test of `load()` methods of `DatasetWriterCacheLoader` and\r\n  `IncrementalDatasetWriterCacheLoader`.\r\n* Fixed a typo\r\n\r\nConflicts:\r\n\tkite-data/kite-data-core/src/main/java/org/kitesdk/data/spi/filesystem/PartitionedDatasetWriter.java"},"comment":{"url":"https://api.github.com/repos/scalingdata/cdk/issues/comments/93593937","html_url":"https://github.com/scalingdata/cdk/pull/13#issuecomment-93593937","issue_url":"https://api.github.com/repos/scalingdata/cdk/issues/13","id":93593937,"user":{"login":"bmccormick","id":696045,"avatar_url":"https://avatars.githubusercontent.com/u/696045?v=3","gravatar_id":"","url":"https://api.github.com/users/bmccormick","html_url":"https://github.com/bmccormick","followers_url":"https://api.github.com/users/bmccormick/followers","following_url":"https://api.github.com/users/bmccormick/following{/other_user}","gists_url":"https://api.github.com/users/bmccormick/gists{/gist_id}","starred_url":"https://api.github.com/users/bmccormick/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bmccormick/subscriptions","organizations_url":"https://api.github.com/users/bmccormick/orgs","repos_url":"https://api.github.com/users/bmccormick/repos","events_url":"https://api.github.com/users/bmccormick/events{/privacy}","received_events_url":"https://api.github.com/users/bmccormick/received_events","type":"User","site_admin":false},"created_at":"2015-04-15T23:11:58Z","updated_at":"2015-04-15T23:11:58Z","body":"+1 LGTM"}}