[ 
https://issues.apache.org/jira/browse/HIVE-1434?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12902611#action_12902611
 ] 

John Sichi commented on HIVE-1434:
----------------------------------

Some points to be resolved.

* I'd like to avoid checking all of the dependency jars into 
cassandra-handler/lib.  From googling around, it sounds like an official 
Cassandra maven repo is not going to happen any time soon, and I'm not sure if 
we can use the unofficial ones.  Would it make sense to just do what we've been 
doing with the Hadoop dependencies, i.e. fetch the tarball via ivy and then 
unpack it?  If so, I can get it added to mirror.facebook.net/facebook/hive-deps.

* Should we attempt to factor out the HBase commonality immediately, or commit 
the overlapping code and then do refactoring as a followup?  I'm fine either 
way; I can give suggestions on how to create the reusable abstract bases and 
where to package+name them.

* Need a checkstyle run to bring the code into conformance there.

* The tests are very skimpy currently; it would be good to add some joins, 
unions, etc.

* There are some minor code cleanups needed; I'll create a review board entry 
and post them there.


> Cassandra Storage Handler
> -------------------------
>
>                 Key: HIVE-1434
>                 URL: https://issues.apache.org/jira/browse/HIVE-1434
>             Project: Hadoop Hive
>          Issue Type: New Feature
>    Affects Versions: 0.7.0
>            Reporter: Edward Capriolo
>            Assignee: Edward Capriolo
>             Fix For: 0.7.0
>
>         Attachments: cas-handle.tar.gz, hive-1434-1.txt, 
> hive-1434-2-patch.txt, hive-1434-3-patch.txt, hive-1434-4-patch.txt
>
>
> Add a cassandra storage handler.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to