{"action":"created","comment":{"url":"https://api.github.com/repos/geotrellis/geotrellis/pulls/comments/87860481","pull_request_review_id":8454514,"id":87860481,"diff_hunk":"@@ -173,33 +173,22 @@ trait ByteReaderExtensions {\n     }\n \n     final def getSignedByteArray(length: Long, valueOffset: Long): Array[Byte] = {\n-      val arr = Array.ofDim[Byte](length.toInt)\n-\n+      // NOTE: We don't support lengths greater than Int.MaxValue yet (or ever).","path":"raster/src/main/scala/geotrellis/raster/io/geotiff/util/ByteReaderExtensions.scala","position":15,"original_position":15,"commit_id":"4b268c4e1840fd24c894d43a2a67a932a3fad821","original_commit_id":"4b268c4e1840fd24c894d43a2a67a932a3fad821","user":{"login":"jbouffard","id":11041248,"avatar_url":"https://avatars.githubusercontent.com/u/11041248?v=3","gravatar_id":"","url":"https://api.github.com/users/jbouffard","html_url":"https://github.com/jbouffard","followers_url":"https://api.github.com/users/jbouffard/followers","following_url":"https://api.github.com/users/jbouffard/following{/other_user}","gists_url":"https://api.github.com/users/jbouffard/gists{/gist_id}","starred_url":"https://api.github.com/users/jbouffard/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jbouffard/subscriptions","organizations_url":"https://api.github.com/users/jbouffard/orgs","repos_url":"https://api.github.com/users/jbouffard/repos","events_url":"https://api.github.com/users/jbouffard/events{/privacy}","received_events_url":"https://api.github.com/users/jbouffard/received_events","type":"User","site_admin":false},"body":"Fixed it. Why it was converted to `Int`? That's because you can't make an `Array.ofDim` using a `Long`.","created_at":"2016-11-14T18:22:24Z","updated_at":"2016-11-14T18:22:24Z","html_url":"https://github.com/geotrellis/geotrellis/pull/1763#discussion_r87860481","pull_request_url":"https://api.github.com/repos/geotrellis/geotrellis/pulls/1763","_links":{"self":{"href":"https://api.github.com/repos/geotrellis/geotrellis/pulls/comments/87860481"},"html":{"href":"https://github.com/geotrellis/geotrellis/pull/1763#discussion_r87860481"},"pull_request":{"href":"https://api.github.com/repos/geotrellis/geotrellis/pulls/1763"}}},"pull_request":{"url":"https://api.github.com/repos/geotrellis/geotrellis/pulls/1763","id":91835880,"html_url":"https://github.com/geotrellis/geotrellis/pull/1763","diff_url":"https://github.com/geotrellis/geotrellis/pull/1763.diff","patch_url":"https://github.com/geotrellis/geotrellis/pull/1763.patch","issue_url":"https://api.github.com/repos/geotrellis/geotrellis/issues/1763","number":1763,"state":"open","locked":false,"title":"Windowed Reading GeoTiffs from S3 and Hdfs","user":{"login":"jbouffard","id":11041248,"avatar_url":"https://avatars.githubusercontent.com/u/11041248?v=3","gravatar_id":"","url":"https://api.github.com/users/jbouffard","html_url":"https://github.com/jbouffard","followers_url":"https://api.github.com/users/jbouffard/followers","following_url":"https://api.github.com/users/jbouffard/following{/other_user}","gists_url":"https://api.github.com/users/jbouffard/gists{/gist_id}","starred_url":"https://api.github.com/users/jbouffard/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jbouffard/subscriptions","organizations_url":"https://api.github.com/users/jbouffard/orgs","repos_url":"https://api.github.com/users/jbouffard/repos","events_url":"https://api.github.com/users/jbouffard/events{/privacy}","received_events_url":"https://api.github.com/users/jbouffard/received_events","type":"User","site_admin":false},"body":"This PR was based on another, preexisting PR: #1758\r\n\r\nTwo new features are added to Geotrellis with this PR: Reading windowed GeoTiffs from S3 and Hdfs. These features will allow for the breaking up of large GeoTiffs into smaller tiles over a distributed system. By using this method, the computation time for each GeoTiff that needs to analyzed can be shortened considerably.\r\n\r\nTo do:\r\n- [x] Unit Testing for Windowed Reading for Hdfs\r\n- [x] Implement Windowed Reading for S3\r\n  - [x] `SpatialGeoTiff`s\r\n  - [x] `TemporalGeoTiff`s\r\n- [x] Unit Testing for Windowed Reading for S3","created_at":"2016-11-01T18:19:19Z","updated_at":"2016-11-14T18:22:24Z","closed_at":null,"merged_at":null,"merge_commit_sha":"b7f7dbf31e3cc2d09a9a334a6ebd9c25f4a8d259","assignee":{"login":"echeipesh","id":1158084,"avatar_url":"https://avatars.githubusercontent.com/u/1158084?v=3","gravatar_id":"","url":"https://api.github.com/users/echeipesh","html_url":"https://github.com/echeipesh","followers_url":"https://api.github.com/users/echeipesh/followers","following_url":"https://api.github.com/users/echeipesh/following{/other_user}","gists_url":"https://api.github.com/users/echeipesh/gists{/gist_id}","starred_url":"https://api.github.com/users/echeipesh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/echeipesh/subscriptions","organizations_url":"https://api.github.com/users/echeipesh/orgs","repos_url":"https://api.github.com/users/echeipesh/repos","events_url":"https://api.github.com/users/echeipesh/events{/privacy}","received_events_url":"https://api.github.com/users/echeipesh/received_events","type":"User","site_admin":false},"assignees":[{"login":"echeipesh","id":1158084,"avatar_url":"https://avatars.githubusercontent.com/u/1158084?v=3","gravatar_id":"","url":"https://api.github.com/users/echeipesh","html_url":"https://github.com/echeipesh","followers_url":"https://api.github.com/users/echeipesh/followers","following_url":"https://api.github.com/users/echeipesh/following{/other_user}","gists_url":"https://api.github.com/users/echeipesh/gists{/gist_id}","starred_url":"https://api.github.com/users/echeipesh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/echeipesh/subscriptions","organizations_url":"https://api.github.com/users/echeipesh/orgs","repos_url":"https://api.github.com/users/echeipesh/repos","events_url":"https://api.github.com/users/echeipesh/events{/privacy}","received_events_url":"https://api.github.com/users/echeipesh/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/geotrellis/geotrellis/milestones/9","html_url":"https://github.com/geotrellis/geotrellis/milestones/1.0","labels_url":"https://api.github.com/repos/geotrellis/geotrellis/milestones/9/labels","id":1626760,"number":9,"title":"1.0","description":"","creator":{"login":"lossyrob","id":2320142,"avatar_url":"https://avatars.githubusercontent.com/u/2320142?v=3","gravatar_id":"","url":"https://api.github.com/users/lossyrob","html_url":"https://github.com/lossyrob","followers_url":"https://api.github.com/users/lossyrob/followers","following_url":"https://api.github.com/users/lossyrob/following{/other_user}","gists_url":"https://api.github.com/users/lossyrob/gists{/gist_id}","starred_url":"https://api.github.com/users/lossyrob/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lossyrob/subscriptions","organizations_url":"https://api.github.com/users/lossyrob/orgs","repos_url":"https://api.github.com/users/lossyrob/repos","events_url":"https://api.github.com/users/lossyrob/events{/privacy}","received_events_url":"https://api.github.com/users/lossyrob/received_events","type":"User","site_admin":false},"open_issues":16,"closed_issues":130,"state":"open","created_at":"2016-03-07T16:39:19Z","updated_at":"2016-11-14T16:50:56Z","due_on":"2016-11-15T05:00:00Z","closed_at":null},"commits_url":"https://api.github.com/repos/geotrellis/geotrellis/pulls/1763/commits","review_comments_url":"https://api.github.com/repos/geotrellis/geotrellis/pulls/1763/comments","review_comment_url":"https://api.github.com/repos/geotrellis/geotrellis/pulls/comments{/number}","comments_url":"https://api.github.com/repos/geotrellis/geotrellis/issues/1763/comments","statuses_url":"https://api.github.com/repos/geotrellis/geotrellis/statuses/4b268c4e1840fd24c894d43a2a67a932a3fad821","head":{"label":"jbouffard:geotiff-rdd","ref":"geotiff-rdd","sha":"4b268c4e1840fd24c894d43a2a67a932a3fad821","user":{"login":"jbouffard","id":11041248,"avatar_url":"https://avatars.githubusercontent.com/u/11041248?v=3","gravatar_id":"","url":"https://api.github.com/users/jbouffard","html_url":"https://github.com/jbouffard","followers_url":"https://api.github.com/users/jbouffard/followers","following_url":"https://api.github.com/users/jbouffard/following{/other_user}","gists_url":"https://api.github.com/users/jbouffard/gists{/gist_id}","starred_url":"https://api.github.com/users/jbouffard/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jbouffard/subscriptions","organizations_url":"https://api.github.com/users/jbouffard/orgs","repos_url":"https://api.github.com/users/jbouffard/repos","events_url":"https://api.github.com/users/jbouffard/events{/privacy}","received_events_url":"https://api.github.com/users/jbouffard/received_events","type":"User","site_admin":false},"repo":{"id":60871739,"name":"geotrellis","full_name":"jbouffard/geotrellis","owner":{"login":"jbouffard","id":11041248,"avatar_url":"https://avatars.githubusercontent.com/u/11041248?v=3","gravatar_id":"","url":"https://api.github.com/users/jbouffard","html_url":"https://github.com/jbouffard","followers_url":"https://api.github.com/users/jbouffard/followers","following_url":"https://api.github.com/users/jbouffard/following{/other_user}","gists_url":"https://api.github.com/users/jbouffard/gists{/gist_id}","starred_url":"https://api.github.com/users/jbouffard/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jbouffard/subscriptions","organizations_url":"https://api.github.com/users/jbouffard/orgs","repos_url":"https://api.github.com/users/jbouffard/repos","events_url":"https://api.github.com/users/jbouffard/events{/privacy}","received_events_url":"https://api.github.com/users/jbouffard/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/jbouffard/geotrellis","description":"GeoTrellis is a geographic data processing engine for high performance applications.","fork":true,"url":"https://api.github.com/repos/jbouffard/geotrellis","forks_url":"https://api.github.com/repos/jbouffard/geotrellis/forks","keys_url":"https://api.github.com/repos/jbouffard/geotrellis/keys{/key_id}","collaborators_url":"https://api.github.com/repos/jbouffard/geotrellis/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/jbouffard/geotrellis/teams","hooks_url":"https://api.github.com/repos/jbouffard/geotrellis/hooks","issue_events_url":"https://api.github.com/repos/jbouffard/geotrellis/issues/events{/number}","events_url":"https://api.github.com/repos/jbouffard/geotrellis/events","assignees_url":"https://api.github.com/repos/jbouffard/geotrellis/assignees{/user}","branches_url":"https://api.github.com/repos/jbouffard/geotrellis/branches{/branch}","tags_url":"https://api.github.com/repos/jbouffard/geotrellis/tags","blobs_url":"https://api.github.com/repos/jbouffard/geotrellis/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/jbouffard/geotrellis/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/jbouffard/geotrellis/git/refs{/sha}","trees_url":"https://api.github.com/repos/jbouffard/geotrellis/git/trees{/sha}","statuses_url":"https://api.github.com/repos/jbouffard/geotrellis/statuses/{sha}","languages_url":"https://api.github.com/repos/jbouffard/geotrellis/languages","stargazers_url":"https://api.github.com/repos/jbouffard/geotrellis/stargazers","contributors_url":"https://api.github.com/repos/jbouffard/geotrellis/contributors","subscribers_url":"https://api.github.com/repos/jbouffard/geotrellis/subscribers","subscription_url":"https://api.github.com/repos/jbouffard/geotrellis/subscription","commits_url":"https://api.github.com/repos/jbouffard/geotrellis/commits{/sha}","git_commits_url":"https://api.github.com/repos/jbouffard/geotrellis/git/commits{/sha}","comments_url":"https://api.github.com/repos/jbouffard/geotrellis/comments{/number}","issue_comment_url":"https://api.github.com/repos/jbouffard/geotrellis/issues/comments{/number}","contents_url":"https://api.github.com/repos/jbouffard/geotrellis/contents/{+path}","compare_url":"https://api.github.com/repos/jbouffard/geotrellis/compare/{base}...{head}","merges_url":"https://api.github.com/repos/jbouffard/geotrellis/merges","archive_url":"https://api.github.com/repos/jbouffard/geotrellis/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/jbouffard/geotrellis/downloads","issues_url":"https://api.github.com/repos/jbouffard/geotrellis/issues{/number}","pulls_url":"https://api.github.com/repos/jbouffard/geotrellis/pulls{/number}","milestones_url":"https://api.github.com/repos/jbouffard/geotrellis/milestones{/number}","notifications_url":"https://api.github.com/repos/jbouffard/geotrellis/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/jbouffard/geotrellis/labels{/name}","releases_url":"https://api.github.com/repos/jbouffard/geotrellis/releases{/id}","deployments_url":"https://api.github.com/repos/jbouffard/geotrellis/deployments","created_at":"2016-06-10T19:23:32Z","updated_at":"2016-06-10T19:23:45Z","pushed_at":"2016-11-14T17:49:44Z","git_url":"git://github.com/jbouffard/geotrellis.git","ssh_url":"git@github.com:jbouffard/geotrellis.git","clone_url":"https://github.com/jbouffard/geotrellis.git","svn_url":"https://github.com/jbouffard/geotrellis","homepage":"http://geotrellis.io","size":279136,"stargazers_count":0,"watchers_count":0,"language":"Scala","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":1,"mirror_url":null,"open_issues_count":2,"forks":1,"open_issues":2,"watchers":0,"default_branch":"master"}},"base":{"label":"geotrellis:master","ref":"master","sha":"720612f419f53e6d0b265bacea849f64244b0c58","user":{"login":"geotrellis","id":3129444,"avatar_url":"https://avatars.githubusercontent.com/u/3129444?v=3","gravatar_id":"","url":"https://api.github.com/users/geotrellis","html_url":"https://github.com/geotrellis","followers_url":"https://api.github.com/users/geotrellis/followers","following_url":"https://api.github.com/users/geotrellis/following{/other_user}","gists_url":"https://api.github.com/users/geotrellis/gists{/gist_id}","starred_url":"https://api.github.com/users/geotrellis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/geotrellis/subscriptions","organizations_url":"https://api.github.com/users/geotrellis/orgs","repos_url":"https://api.github.com/users/geotrellis/repos","events_url":"https://api.github.com/users/geotrellis/events{/privacy}","received_events_url":"https://api.github.com/users/geotrellis/received_events","type":"Organization","site_admin":false},"repo":{"id":3040675,"name":"geotrellis","full_name":"geotrellis/geotrellis","owner":{"login":"geotrellis","id":3129444,"avatar_url":"https://avatars.githubusercontent.com/u/3129444?v=3","gravatar_id":"","url":"https://api.github.com/users/geotrellis","html_url":"https://github.com/geotrellis","followers_url":"https://api.github.com/users/geotrellis/followers","following_url":"https://api.github.com/users/geotrellis/following{/other_user}","gists_url":"https://api.github.com/users/geotrellis/gists{/gist_id}","starred_url":"https://api.github.com/users/geotrellis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/geotrellis/subscriptions","organizations_url":"https://api.github.com/users/geotrellis/orgs","repos_url":"https://api.github.com/users/geotrellis/repos","events_url":"https://api.github.com/users/geotrellis/events{/privacy}","received_events_url":"https://api.github.com/users/geotrellis/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/geotrellis/geotrellis","description":"GeoTrellis is a geographic data processing engine for high performance applications.","fork":false,"url":"https://api.github.com/repos/geotrellis/geotrellis","forks_url":"https://api.github.com/repos/geotrellis/geotrellis/forks","keys_url":"https://api.github.com/repos/geotrellis/geotrellis/keys{/key_id}","collaborators_url":"https://api.github.com/repos/geotrellis/geotrellis/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/geotrellis/geotrellis/teams","hooks_url":"https://api.github.com/repos/geotrellis/geotrellis/hooks","issue_events_url":"https://api.github.com/repos/geotrellis/geotrellis/issues/events{/number}","events_url":"https://api.github.com/repos/geotrellis/geotrellis/events","assignees_url":"https://api.github.com/repos/geotrellis/geotrellis/assignees{/user}","branches_url":"https://api.github.com/repos/geotrellis/geotrellis/branches{/branch}","tags_url":"https://api.github.com/repos/geotrellis/geotrellis/tags","blobs_url":"https://api.github.com/repos/geotrellis/geotrellis/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/geotrellis/geotrellis/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/geotrellis/geotrellis/git/refs{/sha}","trees_url":"https://api.github.com/repos/geotrellis/geotrellis/git/trees{/sha}","statuses_url":"https://api.github.com/repos/geotrellis/geotrellis/statuses/{sha}","languages_url":"https://api.github.com/repos/geotrellis/geotrellis/languages","stargazers_url":"https://api.github.com/repos/geotrellis/geotrellis/stargazers","contributors_url":"https://api.github.com/repos/geotrellis/geotrellis/contributors","subscribers_url":"https://api.github.com/repos/geotrellis/geotrellis/subscribers","subscription_url":"https://api.github.com/repos/geotrellis/geotrellis/subscription","commits_url":"https://api.github.com/repos/geotrellis/geotrellis/commits{/sha}","git_commits_url":"https://api.github.com/repos/geotrellis/geotrellis/git/commits{/sha}","comments_url":"https://api.github.com/repos/geotrellis/geotrellis/comments{/number}","issue_comment_url":"https://api.github.com/repos/geotrellis/geotrellis/issues/comments{/number}","contents_url":"https://api.github.com/repos/geotrellis/geotrellis/contents/{+path}","compare_url":"https://api.github.com/repos/geotrellis/geotrellis/compare/{base}...{head}","merges_url":"https://api.github.com/repos/geotrellis/geotrellis/merges","archive_url":"https://api.github.com/repos/geotrellis/geotrellis/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/geotrellis/geotrellis/downloads","issues_url":"https://api.github.com/repos/geotrellis/geotrellis/issues{/number}","pulls_url":"https://api.github.com/repos/geotrellis/geotrellis/pulls{/number}","milestones_url":"https://api.github.com/repos/geotrellis/geotrellis/milestones{/number}","notifications_url":"https://api.github.com/repos/geotrellis/geotrellis/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/geotrellis/geotrellis/labels{/name}","releases_url":"https://api.github.com/repos/geotrellis/geotrellis/releases{/id}","deployments_url":"https://api.github.com/repos/geotrellis/geotrellis/deployments","created_at":"2011-12-23T14:56:54Z","updated_at":"2016-11-07T17:39:34Z","pushed_at":"2016-11-14T17:57:18Z","git_url":"git://github.com/geotrellis/geotrellis.git","ssh_url":"git@github.com:geotrellis/geotrellis.git","clone_url":"https://github.com/geotrellis/geotrellis.git","svn_url":"https://github.com/geotrellis/geotrellis","homepage":"http://geotrellis.io","size":295634,"stargazers_count":484,"watchers_count":484,"language":"Scala","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":152,"mirror_url":null,"open_issues_count":82,"forks":152,"open_issues":82,"watchers":484,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/geotrellis/geotrellis/pulls/1763"},"html":{"href":"https://github.com/geotrellis/geotrellis/pull/1763"},"issue":{"href":"https://api.github.com/repos/geotrellis/geotrellis/issues/1763"},"comments":{"href":"https://api.github.com/repos/geotrellis/geotrellis/issues/1763/comments"},"review_comments":{"href":"https://api.github.com/repos/geotrellis/geotrellis/pulls/1763/comments"},"review_comment":{"href":"https://api.github.com/repos/geotrellis/geotrellis/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/geotrellis/geotrellis/pulls/1763/commits"},"statuses":{"href":"https://api.github.com/repos/geotrellis/geotrellis/statuses/4b268c4e1840fd24c894d43a2a67a932a3fad821"}}}}
{"actor":{"display_login":"jbouffard"}}