You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Dave Brosius (Created) (JIRA)" <ji...@apache.org> on 2012/02/11 04:21:06 UTC

[jira] [Created] (CASSANDRA-3890) [patch] add clientutil class for building composite column names

[patch] add clientutil class for building composite column names
----------------------------------------------------------------

                 Key: CASSANDRA-3890
                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3890
             Project: Cassandra
          Issue Type: Improvement
          Components: Tools
            Reporter: Dave Brosius
            Priority: Trivial


allow for clients do do

CompositeNameBuilder builder = new CompositeNameBuilder();
builder.append("US");
builder.append(91210);
builder.append("San Mateo");
Column col = new Column(builder.getComposite());

BytesType, UTF8Type, LongType, LexicalUUIDType, TimeUUIDType are directly supported, AsciiType and custom types are supported thru a append(byte... value) catchall.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (CASSANDRA-3890) [patch] add clientutil class for building composite column names

Posted by "Dave Brosius (Updated) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/CASSANDRA-3890?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Dave Brosius updated CASSANDRA-3890:
------------------------------------

    Attachment: add_composite_builder.diff
    
> [patch] add clientutil class for building composite column names
> ----------------------------------------------------------------
>
>                 Key: CASSANDRA-3890
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3890
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Tools
>            Reporter: Dave Brosius
>            Priority: Trivial
>         Attachments: add_composite_builder.diff
>
>
> allow for clients do do
> CompositeNameBuilder builder = new CompositeNameBuilder();
> builder.append("US");
> builder.append(91210);
> builder.append("San Mateo");
> Column col = new Column(builder.getComposite());
> BytesType, UTF8Type, LongType, LexicalUUIDType, TimeUUIDType are directly supported, AsciiType and custom types are supported thru a append(byte... value) catchall.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (CASSANDRA-3890) [patch] add clientutil class for building composite column names

Posted by "Dave Brosius (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/CASSANDRA-3890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13497683#comment-13497683 ] 

Dave Brosius commented on CASSANDRA-3890:
-----------------------------------------

It would theoretically be nice to add CompositeType to client-utils jar as building composite columns in thrift is tedious presently, but given that higher level libs probably have this functionality, and also the dependency problems that already exist in client-utils, probably isn't worth it.
                
> [patch] add clientutil class for building composite column names
> ----------------------------------------------------------------
>
>                 Key: CASSANDRA-3890
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3890
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Tools
>            Reporter: Dave Brosius
>            Priority: Trivial
>         Attachments: add_composite_builder.diff
>
>
> allow for clients do do
> CompositeNameBuilder builder = new CompositeNameBuilder();
> builder.append("US");
> builder.append(91210);
> builder.append("San Mateo");
> Column col = new Column(builder.getComposite());
> BytesType, UTF8Type, LongType, LexicalUUIDType, TimeUUIDType are directly supported, AsciiType and custom types are supported thru a append(byte... value) catchall.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Commented] (CASSANDRA-3890) [patch] add clientutil class for building composite column names

Posted by "Sylvain Lebresne (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/CASSANDRA-3890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13206876#comment-13206876 ] 

Sylvain Lebresne commented on CASSANDRA-3890:
---------------------------------------------

With CQL 3, we've added CompositeType.Builder (so in CompositeType.java) that does essentially the same thing. Could be worth including that instead of duplicating the code.
                
> [patch] add clientutil class for building composite column names
> ----------------------------------------------------------------
>
>                 Key: CASSANDRA-3890
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3890
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Tools
>            Reporter: Dave Brosius
>            Priority: Trivial
>         Attachments: add_composite_builder.diff
>
>
> allow for clients do do
> CompositeNameBuilder builder = new CompositeNameBuilder();
> builder.append("US");
> builder.append(91210);
> builder.append("San Mateo");
> Column col = new Column(builder.getComposite());
> BytesType, UTF8Type, LongType, LexicalUUIDType, TimeUUIDType are directly supported, AsciiType and custom types are supported thru a append(byte... value) catchall.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (CASSANDRA-3890) [patch] add clientutil class for building composite column names

Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/CASSANDRA-3890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13497632#comment-13497632 ] 

Jonathan Ellis commented on CASSANDRA-3890:
-------------------------------------------

Is this still a concern?
                
> [patch] add clientutil class for building composite column names
> ----------------------------------------------------------------
>
>                 Key: CASSANDRA-3890
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3890
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Tools
>            Reporter: Dave Brosius
>            Priority: Trivial
>         Attachments: add_composite_builder.diff
>
>
> allow for clients do do
> CompositeNameBuilder builder = new CompositeNameBuilder();
> builder.append("US");
> builder.append(91210);
> builder.append("San Mateo");
> Column col = new Column(builder.getComposite());
> BytesType, UTF8Type, LongType, LexicalUUIDType, TimeUUIDType are directly supported, AsciiType and custom types are supported thru a append(byte... value) catchall.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Commented] (CASSANDRA-3890) [patch] add clientutil class for building composite column names

Posted by "Dave Brosius (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/CASSANDRA-3890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13207415#comment-13207415 ] 

Dave Brosius commented on CASSANDRA-3890:
-----------------------------------------

Sure, thats fine, but it should probably be broken out of CompositeType and moved to packages used by clientutils for consistency. (Didn't see it there, hiding among the java.management classes :)
                
> [patch] add clientutil class for building composite column names
> ----------------------------------------------------------------
>
>                 Key: CASSANDRA-3890
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3890
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Tools
>            Reporter: Dave Brosius
>            Priority: Trivial
>         Attachments: add_composite_builder.diff
>
>
> allow for clients do do
> CompositeNameBuilder builder = new CompositeNameBuilder();
> builder.append("US");
> builder.append(91210);
> builder.append("San Mateo");
> Column col = new Column(builder.getComposite());
> BytesType, UTF8Type, LongType, LexicalUUIDType, TimeUUIDType are directly supported, AsciiType and custom types are supported thru a append(byte... value) catchall.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira