cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Mike Spertus (JIRA)" <>
Subject [jira] [Commented] (CASSANDRA-6420) CassandraStorage should not assume all DataBags are DefaultDataBags
Date Mon, 23 Dec 2013 05:25:50 GMT


Mike Spertus commented on CASSANDRA-6420:

Thanks, Brandon!

> CassandraStorage should not assume all DataBags are DefaultDataBags
> -------------------------------------------------------------------
>                 Key: CASSANDRA-6420
>                 URL:
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Hadoop
>         Environment: All environments
>            Reporter: Mike Spertus
>            Assignee: Mike Spertus
>              Labels: pig
>             Fix For: 1.2.14, 2.0.4
>         Attachments: patch.txt
>   Original Estimate: 1h
>  Remaining Estimate: 1h
> CassandraStorage improperly assumes all DataBags are DefaultDataBags. As a result, natural
Pig code can't be used with CassandraStorage. For example:
> {quote}
> {{STORE B into  'cassandra://MyKeySpace/MyColumnFamily' using CassandraStorage();}}
> {quote}
> fails with a complaint that a {{NonSpillableDataBag}} can't be converted into a {{DefaultDataBag}}.
> Since the {{CassandraStorage}} code only calls methods from {{DataBag}}, there is no
need for this artifical restriction. After applying the attached patch, the above code works
fine, making CassandraStorage much easier to use.
> This is my first submission to Cassandra, so I apologize for any incorrect process. Please
let me know what I should do differently. In particular, I am a little unclear where I should
put the test. I am thinking I should put it in Is this correct
or should it be somewhere else? I'll create a test as soon as I understand. 

This message was sent by Atlassian JIRA

View raw message