{"action":"created","issue":{"url":"https://api.github.com/repos/bigdatagenomics/adam/issues/1293","repository_url":"https://api.github.com/repos/bigdatagenomics/adam","labels_url":"https://api.github.com/repos/bigdatagenomics/adam/issues/1293/labels{/name}","comments_url":"https://api.github.com/repos/bigdatagenomics/adam/issues/1293/comments","events_url":"https://api.github.com/repos/bigdatagenomics/adam/issues/1293/events","html_url":"https://github.com/bigdatagenomics/adam/pull/1293","id":191886522,"number":1293,"title":"HBase as a separate repo","user":{"login":"jpdna","id":7423296,"avatar_url":"https://avatars.githubusercontent.com/u/7423296?v=3","gravatar_id":"","url":"https://api.github.com/users/jpdna","html_url":"https://github.com/jpdna","followers_url":"https://api.github.com/users/jpdna/followers","following_url":"https://api.github.com/users/jpdna/following{/other_user}","gists_url":"https://api.github.com/users/jpdna/gists{/gist_id}","starred_url":"https://api.github.com/users/jpdna/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jpdna/subscriptions","organizations_url":"https://api.github.com/users/jpdna/orgs","repos_url":"https://api.github.com/users/jpdna/repos","events_url":"https://api.github.com/users/jpdna/events{/privacy}","received_events_url":"https://api.github.com/users/jpdna/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-11-27T21:22:48Z","updated_at":"2016-11-28T18:18:34Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/bigdatagenomics/adam/pulls/1293","html_url":"https://github.com/bigdatagenomics/adam/pull/1293","diff_url":"https://github.com/bigdatagenomics/adam/pull/1293.diff","patch_url":"https://github.com/bigdatagenomics/adam/pull/1293.patch"},"body":"Requires: https://github.com/jpdna/bdghbase commit  809c554\r\n\r\nThis PR is a stub for any changes to ADAM to make use of HBase code as a separate repo, and discussion of that approach.\r\n\r\nI decided to try splitting out the HBase code into a different repo, bdghbase above.\r\nThe motivation I see for this are:\r\n\r\n1) We may have a number of different data backends including HBase -  Kudu for one.  It may be better not to put these all into the main ADAM repo.\r\n\r\n2) I wasn't able to yet find a way to get \"provided\" dependency for hbase library code to work, so it will be bloating the compiled hbase module for now , better to keep that out of main ADAM, even if its working fine.\r\n\r\n3) Testing with CI is going to be a pain for hbase, for the moment tests are going to require that an hbase instance is accessible, as I can't seem to get the mock \"mini\" hbase cluster to work.  Thus this complicates testing and CI, so better to keep that complexity in separate repo.\r\n\r\n4) nice to have fast compilation time for the hbase code under development\r\n\r\nChoice of separate repo or not seems irrelevant to how adam-shell or downstream applications using ADAM as a library would be built, I can interact with ADAM and hbase from adam-shell just as before using this PR.   \r\n\r\n- I plan to add a \"hbase\" profile that will turn of the hbase dependencies in ADAM, sound good?\r\n\r\n- I will clearly have to publish bdghbase to maven was currently do for bdg-utils and other seperate bdg repos.\r\n\r\n- I'm not sure how to handle the CLI in this instance for hbase.  We could include the vcf2Hbase CLI code as it currently is in this PR, and perhaps it would only work in the -hbase profile was turned on.  We'd want the command and cli help to also be different based on the profile.\r\nThis doesn't seem ideal to me though - if bdghbase (or bdgkudu) is to be a separate repo, I feel like the CLI code should be in that repo as well, but I am not sure how to best integrate it with the current ADAM CLI, thoughts?\r\n\r\nFor now I am looking for general comments on this approach.  I'll ask later for review of the hbase code, as I am nearly done addressing the comments in: #1246\r\n\r\n\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/bigdatagenomics/adam/issues/comments/263349467","html_url":"https://github.com/bigdatagenomics/adam/pull/1293#issuecomment-263349467","issue_url":"https://api.github.com/repos/bigdatagenomics/adam/issues/1293","id":263349467,"user":{"login":"heuermh","id":35293,"avatar_url":"https://avatars.githubusercontent.com/u/35293?v=3","gravatar_id":"","url":"https://api.github.com/users/heuermh","html_url":"https://github.com/heuermh","followers_url":"https://api.github.com/users/heuermh/followers","following_url":"https://api.github.com/users/heuermh/following{/other_user}","gists_url":"https://api.github.com/users/heuermh/gists{/gist_id}","starred_url":"https://api.github.com/users/heuermh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/heuermh/subscriptions","organizations_url":"https://api.github.com/users/heuermh/orgs","repos_url":"https://api.github.com/users/heuermh/repos","events_url":"https://api.github.com/users/heuermh/events{/privacy}","received_events_url":"https://api.github.com/users/heuermh/received_events","type":"User","site_admin":false},"created_at":"2016-11-28T18:18:34Z","updated_at":"2016-11-28T18:18:34Z","body":"Extending the ADAM command line interface from an external repository is demonstrated at https://github.com/heuermh/adam-commands.  Not necessarily saying that is the right way to go for this, we might need to discuss the overall approach in person."}}
{"actor":{"display_login":"heuermh"}}