You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Jonathan Ellis (JIRA)" <ji...@apache.org> on 2010/10/05 23:16:50 UTC

[jira] Created: (CASSANDRA-1578) Add support for column and row validators to cli

Add support for column and row validators to cli
------------------------------------------------

                 Key: CASSANDRA-1578
                 URL: https://issues.apache.org/jira/browse/CASSANDRA-1578
             Project: Cassandra
          Issue Type: New Feature
          Components: Tools
            Reporter: Jonathan Ellis
            Assignee: Pavel Yaskevich
             Fix For: 0.7.1


Similar to CASSANDRA-1523, but this applies to column values where 1523 applied to column names.  We want to parse/display column values in human-readable format.

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


[jira] Commented: (CASSANDRA-1578) Add support for column and row validators to cli

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

Jonathan Ellis commented on CASSANDRA-1578:
-------------------------------------------

getValidatorForValue needs to check for a CF-level validator if there isn't one on the column

> Add support for column and row validators to cli
> ------------------------------------------------
>
>                 Key: CASSANDRA-1578
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-1578
>             Project: Cassandra
>          Issue Type: New Feature
>          Components: Tools
>            Reporter: Jonathan Ellis
>            Assignee: Pavel Yaskevich
>             Fix For: 0.7.1
>
>         Attachments: CASSANDRA-1578.patch
>
>
> Similar to CASSANDRA-1523, but this applies to column values where 1523 applied to column names.  We want to parse/display column values in human-readable format.

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


[jira] Updated: (CASSANDRA-1578) Add support for column and row validators to cli

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

Pavel Yaskevich updated CASSANDRA-1578:
---------------------------------------

    Attachment: CASSANDRA-1578-v2.patch

getValidatorForValue method tries to use CfDef.default_validation_class if column validator wasn't found.

> Add support for column and row validators to cli
> ------------------------------------------------
>
>                 Key: CASSANDRA-1578
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-1578
>             Project: Cassandra
>          Issue Type: New Feature
>          Components: Tools
>            Reporter: Jonathan Ellis
>            Assignee: Pavel Yaskevich
>             Fix For: 0.7.1
>
>         Attachments: CASSANDRA-1578-v2.patch, CASSANDRA-1578.patch
>
>
> Similar to CASSANDRA-1523, but this applies to column values where 1523 applied to column names.  We want to parse/display column values in human-readable format.

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


[jira] Updated: (CASSANDRA-1578) Add support for column and row validators to cli

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

Pavel Yaskevich updated CASSANDRA-1578:
---------------------------------------

    Attachment: CASSANDRA-1578.patch

> Add support for column and row validators to cli
> ------------------------------------------------
>
>                 Key: CASSANDRA-1578
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-1578
>             Project: Cassandra
>          Issue Type: New Feature
>          Components: Tools
>            Reporter: Jonathan Ellis
>            Assignee: Pavel Yaskevich
>             Fix For: 0.7.1
>
>         Attachments: CASSANDRA-1578.patch
>
>
> Similar to CASSANDRA-1523, but this applies to column values where 1523 applied to column names.  We want to parse/display column values in human-readable format.

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


[jira] Commented: (CASSANDRA-1578) Add support for column and row validators to cli

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

Jonathan Ellis commented on CASSANDRA-1578:
-------------------------------------------

you can annotate specific columns with metadata specifying what type of data should be allowed, and optionally declaring an index on it (see CASSANDRA-1583).  if we know that a column value is LongType, we should convert the byte[] into a Long before displaying, and vice versa for accepting input.  Right now we are just assuming all values are Strings:

{code}
        // print results
        css_.out.printf("=> (column=%s, value=%s, timestamp=%d)\n",
                        formatColumnName(keySpace, columnFamily, column), new String(column.value, "UTF-8"), column.timestamp);
{code}

{code}
        // do the insert
        thriftClient_.insert(key.getBytes(), new ColumnParent(columnFamily).setSuper_column(superColumnName),
                             new Column(columnNameAsByteArray(columnName, columnFamily), value.getBytes(), FBUtilities.timestampMicros()), ConsistencyLevel.ONE);
{code}

> Add support for column and row validators to cli
> ------------------------------------------------
>
>                 Key: CASSANDRA-1578
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-1578
>             Project: Cassandra
>          Issue Type: New Feature
>          Components: Tools
>            Reporter: Jonathan Ellis
>            Assignee: Pavel Yaskevich
>             Fix For: 0.7.1
>
>
> Similar to CASSANDRA-1523, but this applies to column values where 1523 applied to column names.  We want to parse/display column values in human-readable format.

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


[jira] Commented: (CASSANDRA-1578) Add support for column and row validators to cli

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

Pavel Yaskevich commented on CASSANDRA-1578:
--------------------------------------------

Can you please provide a better description? What I have right now is that skipping of unnecessary quoting should be removed from values...

> Add support for column and row validators to cli
> ------------------------------------------------
>
>                 Key: CASSANDRA-1578
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-1578
>             Project: Cassandra
>          Issue Type: New Feature
>          Components: Tools
>            Reporter: Jonathan Ellis
>            Assignee: Pavel Yaskevich
>             Fix For: 0.7.1
>
>
> Similar to CASSANDRA-1523, but this applies to column values where 1523 applied to column names.  We want to parse/display column values in human-readable format.

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


[jira] Updated: (CASSANDRA-1578) Add support for column and row validators to cli

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

Pavel Yaskevich updated CASSANDRA-1578:
---------------------------------------

    Reviewer: jbellis  (was: jb)

> Add support for column and row validators to cli
> ------------------------------------------------
>
>                 Key: CASSANDRA-1578
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-1578
>             Project: Cassandra
>          Issue Type: New Feature
>          Components: Tools
>            Reporter: Jonathan Ellis
>            Assignee: Pavel Yaskevich
>             Fix For: 0.7.1
>
>         Attachments: CASSANDRA-1578.patch
>
>
> Similar to CASSANDRA-1523, but this applies to column values where 1523 applied to column names.  We want to parse/display column values in human-readable format.

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


[jira] Commented: (CASSANDRA-1578) Add support for column and row validators to cli

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

Hudson commented on CASSANDRA-1578:
-----------------------------------

Integrated in Cassandra #563 (See [https://hudson.apache.org/hudson/job/Cassandra/563/])
    

> Add support for column and row validators to cli
> ------------------------------------------------
>
>                 Key: CASSANDRA-1578
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-1578
>             Project: Cassandra
>          Issue Type: New Feature
>          Components: Tools
>            Reporter: Jonathan Ellis
>            Assignee: Pavel Yaskevich
>             Fix For: 0.7.1
>
>         Attachments: CASSANDRA-1578-v2.patch, CASSANDRA-1578.patch
>
>
> Similar to CASSANDRA-1523, but this applies to column values where 1523 applied to column names.  We want to parse/display column values in human-readable format.

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