You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Jason Rutherglen (JIRA)" <ji...@apache.org> on 2011/09/07 04:05:09 UTC

[jira] [Created] (CASSANDRA-3147) Secondary index API methods need to throw exceptions

Secondary index API methods need to throw exceptions
----------------------------------------------------

                 Key: CASSANDRA-3147
                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3147
             Project: Cassandra
          Issue Type: Improvement
          Components: Core
            Reporter: Jason Rutherglen
            Assignee: Jason Rutherglen
            Priority: Trivial
             Fix For: 1.0


The new secondary index API needs to throw IOException on some of the methods (where they're missing).

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (CASSANDRA-3147) Add rowlevel support to secondary index API and methods need to throw exceptions

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

Jason Rutherglen commented on CASSANDRA-3147:
---------------------------------------------

Jake, I think we can do a polymorphic base class.  Then create two different kinds of index interfaces, row level and column level.  The SecondaryIndexManager will then do an instance of.

> Add rowlevel support to secondary index API and methods need to throw exceptions
> --------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-3147
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3147
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Jason Rutherglen
>            Assignee: Jason Rutherglen
>            Priority: Minor
>             Fix For: 1.0.0
>
>         Attachments: v1-0001-CASSANDRA-3147-add-rowLevel-logic-and-exceptions.txt
>
>
> The new secondary index API needs to throw IOException on some of the methods (where they're missing).
> For index types that index an entire row (like lucene) we need to change the api to handle one instance of the index per ColumnFamily.  This can be done by adding a isRowLevelIndex() method.  When the index manager sees this it will add all columns of the same index class to the same index instance.  and on search it will send the entire row to the class vs updating each column.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (CASSANDRA-3147) Add rowlevel support to secondary index API and methods need to throw exceptions

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

T Jake Luciani updated CASSANDRA-3147:
--------------------------------------

    Attachment:     (was: v1-0001-CASSANDRA-3147-add-rowLevel-logic-and-exceptions.txt)

> Add rowlevel support to secondary index API and methods need to throw exceptions
> --------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-3147
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3147
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Jason Rutherglen
>            Assignee: Jason Rutherglen
>            Priority: Minor
>             Fix For: 1.0.0
>
>         Attachments: v1-0001-CASSANDRA-3147-add-rowLevel-logic-and-exceptions.txt
>
>
> The new secondary index API needs to throw IOException on some of the methods (where they're missing).
> For index types that index an entire row (like lucene) we need to change the api to handle one instance of the index per ColumnFamily.  This can be done by adding a isRowLevelIndex() method.  When the index manager sees this it will add all columns of the same index class to the same index instance.  and on search it will send the entire row to the class vs updating each column.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Resolved] (CASSANDRA-3147) Add rowlevel support to secondary index API and methods need to throw exceptions

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

Pavel Yaskevich resolved CASSANDRA-3147.
----------------------------------------

    Resolution: Fixed

Committed with few method name changes such as getSystemTableName changed to getNameForSystemTable, getColumnDef to getColumnDefs etc.

> Add rowlevel support to secondary index API and methods need to throw exceptions
> --------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-3147
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3147
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Jason Rutherglen
>            Assignee: Jason Rutherglen
>            Priority: Minor
>             Fix For: 1.0.0
>
>         Attachments: v1-0001-CASSANDRA-3147-add-rowLevel-logic-and-exceptions.txt
>
>
> The new secondary index API needs to throw IOException on some of the methods (where they're missing).
> For index types that index an entire row (like lucene) we need to change the api to handle one instance of the index per ColumnFamily.  This can be done by adding a PerColumnSecondary and PerRowSecondaryIndex base classes.  When the index manager sees this it will add all columns of the same index class to the same index instance.  and on search it will send the entire row to the class vs updating each column.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (CASSANDRA-3147) Secondary index API methods need to throw exceptions

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

T Jake Luciani updated CASSANDRA-3147:
--------------------------------------

    Attachment: v1-0001-CASSANDRA-3147-add-rowLevel-logic-and-exceptions.txt

> Secondary index API methods need to throw exceptions
> ----------------------------------------------------
>
>                 Key: CASSANDRA-3147
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3147
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Jason Rutherglen
>            Assignee: Jason Rutherglen
>            Priority: Trivial
>             Fix For: 1.0.0
>
>         Attachments: v1-0001-CASSANDRA-3147-add-rowLevel-logic-and-exceptions.txt
>
>
> The new secondary index API needs to throw IOException on some of the methods (where they're missing).

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (CASSANDRA-3147) Add rowlevel support to secondary index API and methods need to throw exceptions

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

T Jake Luciani updated CASSANDRA-3147:
--------------------------------------

       Reviewer: xedin
    Description: 
The new secondary index API needs to throw IOException on some of the methods (where they're missing).

For index types that index an entire row (like lucene) we need to change the api to handle one instance of the index per ColumnFamily.  This can be done by adding a PerColumnSecondary and PerRowSecondaryIndex base classes.  When the index manager sees this it will add all columns of the same index class to the same index instance.  and on search it will send the entire row to the class vs updating each column.

  was:
The new secondary index API needs to throw IOException on some of the methods (where they're missing).

For index types that index an entire row (like lucene) we need to change the api to handle one instance of the index per ColumnFamily.  This can be done by adding a isRowLevelIndex() method.  When the index manager sees this it will add all columns of the same index class to the same index instance.  and on search it will send the entire row to the class vs updating each column.


> Add rowlevel support to secondary index API and methods need to throw exceptions
> --------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-3147
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3147
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Jason Rutherglen
>            Assignee: Jason Rutherglen
>            Priority: Minor
>             Fix For: 1.0.0
>
>         Attachments: v1-0001-CASSANDRA-3147-add-rowLevel-logic-and-exceptions.txt
>
>
> The new secondary index API needs to throw IOException on some of the methods (where they're missing).
> For index types that index an entire row (like lucene) we need to change the api to handle one instance of the index per ColumnFamily.  This can be done by adding a PerColumnSecondary and PerRowSecondaryIndex base classes.  When the index manager sees this it will add all columns of the same index class to the same index instance.  and on search it will send the entire row to the class vs updating each column.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (CASSANDRA-3147) Add rowlevel support to secondary index API and methods need to throw exceptions

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

T Jake Luciani updated CASSANDRA-3147:
--------------------------------------

    Description: 
The new secondary index API needs to throw IOException on some of the methods (where they're missing).

For index types that index an entire row (like lucene) we need to change the api to handle one instance of the index per ColumnFamily.  This can be done by adding a isRowLevelIndex() method.  When the index manager sees this it will add all columns of the same index class to the same index instance.  and on search it will send the entire row to the class vs updating each column.

  was:The new secondary index API needs to throw IOException on some of the methods (where they're missing).

       Priority: Minor  (was: Trivial)
        Summary: Add rowlevel support to secondary index API and methods need to throw exceptions  (was: Secondary index API methods need to throw exceptions)

> Add rowlevel support to secondary index API and methods need to throw exceptions
> --------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-3147
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3147
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Jason Rutherglen
>            Assignee: Jason Rutherglen
>            Priority: Minor
>             Fix For: 1.0.0
>
>         Attachments: v1-0001-CASSANDRA-3147-add-rowLevel-logic-and-exceptions.txt
>
>
> The new secondary index API needs to throw IOException on some of the methods (where they're missing).
> For index types that index an entire row (like lucene) we need to change the api to handle one instance of the index per ColumnFamily.  This can be done by adding a isRowLevelIndex() method.  When the index manager sees this it will add all columns of the same index class to the same index instance.  and on search it will send the entire row to the class vs updating each column.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (CASSANDRA-3147) Add rowlevel support to secondary index API and methods need to throw exceptions

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

Jason Rutherglen commented on CASSANDRA-3147:
---------------------------------------------

I'll post a patch shortly.

> Add rowlevel support to secondary index API and methods need to throw exceptions
> --------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-3147
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3147
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Jason Rutherglen
>            Assignee: Jason Rutherglen
>            Priority: Minor
>             Fix For: 1.0.0
>
>         Attachments: v1-0001-CASSANDRA-3147-add-rowLevel-logic-and-exceptions.txt
>
>
> The new secondary index API needs to throw IOException on some of the methods (where they're missing).
> For index types that index an entire row (like lucene) we need to change the api to handle one instance of the index per ColumnFamily.  This can be done by adding a isRowLevelIndex() method.  When the index manager sees this it will add all columns of the same index class to the same index instance.  and on search it will send the entire row to the class vs updating each column.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (CASSANDRA-3147) Add rowlevel support to secondary index API and methods need to throw exceptions

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

T Jake Luciani commented on CASSANDRA-3147:
-------------------------------------------

Posted patch with PerRow and PerColumn SecondaryIndex classes

> Add rowlevel support to secondary index API and methods need to throw exceptions
> --------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-3147
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3147
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Jason Rutherglen
>            Assignee: Jason Rutherglen
>            Priority: Minor
>             Fix For: 1.0.0
>
>         Attachments: v1-0001-CASSANDRA-3147-add-rowLevel-logic-and-exceptions.txt
>
>
> The new secondary index API needs to throw IOException on some of the methods (where they're missing).
> For index types that index an entire row (like lucene) we need to change the api to handle one instance of the index per ColumnFamily.  This can be done by adding a PerColumnSecondary and PerRowSecondaryIndex base classes.  When the index manager sees this it will add all columns of the same index class to the same index instance.  and on search it will send the entire row to the class vs updating each column.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (CASSANDRA-3147) Add rowlevel support to secondary index API and methods need to throw exceptions

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

T Jake Luciani updated CASSANDRA-3147:
--------------------------------------

    Attachment: v1-0001-CASSANDRA-3147-add-rowLevel-logic-and-exceptions.txt

> Add rowlevel support to secondary index API and methods need to throw exceptions
> --------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-3147
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3147
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Jason Rutherglen
>            Assignee: Jason Rutherglen
>            Priority: Minor
>             Fix For: 1.0.0
>
>         Attachments: v1-0001-CASSANDRA-3147-add-rowLevel-logic-and-exceptions.txt, v1-0001-CASSANDRA-3147-add-rowLevel-logic-and-exceptions.txt
>
>
> The new secondary index API needs to throw IOException on some of the methods (where they're missing).
> For index types that index an entire row (like lucene) we need to change the api to handle one instance of the index per ColumnFamily.  This can be done by adding a isRowLevelIndex() method.  When the index manager sees this it will add all columns of the same index class to the same index instance.  and on search it will send the entire row to the class vs updating each column.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (CASSANDRA-3147) Add rowlevel support to secondary index API and methods need to throw exceptions

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

T Jake Luciani updated CASSANDRA-3147:
--------------------------------------

    Attachment:     (was: v1-0001-CASSANDRA-3147-add-rowLevel-logic-and-exceptions.txt)

> Add rowlevel support to secondary index API and methods need to throw exceptions
> --------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-3147
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3147
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Jason Rutherglen
>            Assignee: Jason Rutherglen
>            Priority: Minor
>             Fix For: 1.0.0
>
>         Attachments: v1-0001-CASSANDRA-3147-add-rowLevel-logic-and-exceptions.txt
>
>
> The new secondary index API needs to throw IOException on some of the methods (where they're missing).
> For index types that index an entire row (like lucene) we need to change the api to handle one instance of the index per ColumnFamily.  This can be done by adding a PerColumnSecondary and PerRowSecondaryIndex base classes.  When the index manager sees this it will add all columns of the same index class to the same index instance.  and on search it will send the entire row to the class vs updating each column.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Assigned] (CASSANDRA-3147) Add rowlevel support to secondary index API and methods need to throw exceptions

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

T Jake Luciani reassigned CASSANDRA-3147:
-----------------------------------------

    Assignee: T Jake Luciani  (was: Jason Rutherglen)

> Add rowlevel support to secondary index API and methods need to throw exceptions
> --------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-3147
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3147
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Jason Rutherglen
>            Assignee: T Jake Luciani
>            Priority: Minor
>             Fix For: 1.0.0
>
>         Attachments: v1-0001-CASSANDRA-3147-add-rowLevel-logic-and-exceptions.txt
>
>
> The new secondary index API needs to throw IOException on some of the methods (where they're missing).
> For index types that index an entire row (like lucene) we need to change the api to handle one instance of the index per ColumnFamily.  This can be done by adding a PerColumnSecondary and PerRowSecondaryIndex base classes.  When the index manager sees this it will add all columns of the same index class to the same index instance.  and on search it will send the entire row to the class vs updating each column.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (CASSANDRA-3147) Add rowlevel support to secondary index API and methods need to throw exceptions

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

T Jake Luciani updated CASSANDRA-3147:
--------------------------------------

    Attachment: v1-0001-CASSANDRA-3147-add-rowLevel-logic-and-exceptions.txt

> Add rowlevel support to secondary index API and methods need to throw exceptions
> --------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-3147
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3147
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Jason Rutherglen
>            Assignee: Jason Rutherglen
>            Priority: Minor
>             Fix For: 1.0.0
>
>         Attachments: v1-0001-CASSANDRA-3147-add-rowLevel-logic-and-exceptions.txt, v1-0001-CASSANDRA-3147-add-rowLevel-logic-and-exceptions.txt
>
>
> The new secondary index API needs to throw IOException on some of the methods (where they're missing).
> For index types that index an entire row (like lucene) we need to change the api to handle one instance of the index per ColumnFamily.  This can be done by adding a PerColumnSecondary and PerRowSecondaryIndex base classes.  When the index manager sees this it will add all columns of the same index class to the same index instance.  and on search it will send the entire row to the class vs updating each column.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira