You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Gary Dusbabek (JIRA)" <ji...@apache.org> on 2010/08/03 23:31:18 UTC

[jira] Created: (CASSANDRA-1354) Static CFMetaData objects are using the wrong constructor.

Static CFMetaData objects are using the wrong constructor.
----------------------------------------------------------

                 Key: CASSANDRA-1354
                 URL: https://issues.apache.org/jira/browse/CASSANDRA-1354
             Project: Cassandra
          Issue Type: Bug
            Reporter: Gary Dusbabek
            Assignee: Gary Dusbabek
             Fix For: 0.7 beta 1


This means they are getting assigned ids > 1000.  Static CFMs should be using the private constructor that uses a specific cfid.

I'm pretty sure they just need to be getting a readRepairChance=0.

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


[jira] Commented: (CASSANDRA-1354) Static CFMetaData objects are using the wrong constructor.

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

Jonathan Ellis commented on CASSANDRA-1354:
-------------------------------------------

+1, but for the record, what badness happens when the static CFs get part of the "public pool" of ids?

> Static CFMetaData objects are using the wrong constructor.
> ----------------------------------------------------------
>
>                 Key: CASSANDRA-1354
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-1354
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Gary Dusbabek
>            Assignee: Gary Dusbabek
>             Fix For: 0.7 beta 1
>
>         Attachments: 0001-use-correct-constructor-when-creating-static-CFM-ins.patch
>
>
> This means they are getting assigned ids > 1000.  Static CFMs should be using the private constructor that uses a specific cfid that is <1000
> I'm pretty sure they just need to be getting a readRepairChance=0.

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


[jira] Commented: (CASSANDRA-1354) Static CFMetaData objects are using the wrong constructor.

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

Gary Dusbabek commented on CASSANDRA-1354:
------------------------------------------

The next time we create a static CFM (like we did with StatisticsCf) it would use an ID that is already used by a regular column family that just hasn't been loaded yet.  

> Static CFMetaData objects are using the wrong constructor.
> ----------------------------------------------------------
>
>                 Key: CASSANDRA-1354
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-1354
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Gary Dusbabek
>            Assignee: Gary Dusbabek
>             Fix For: 0.7 beta 1
>
>         Attachments: 0001-use-correct-constructor-when-creating-static-CFM-ins.patch
>
>
> This means they are getting assigned ids > 1000.  Static CFMs should be using the private constructor that uses a specific cfid that is <1000
> I'm pretty sure they just need to be getting a readRepairChance=0.

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


[jira] Updated: (CASSANDRA-1354) Static CFMetaData objects are using the wrong constructor.

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

Gary Dusbabek updated CASSANDRA-1354:
-------------------------------------

    Attachment: 0001-use-correct-constructor-when-creating-static-CFM-ins.patch

> Static CFMetaData objects are using the wrong constructor.
> ----------------------------------------------------------
>
>                 Key: CASSANDRA-1354
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-1354
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Gary Dusbabek
>            Assignee: Gary Dusbabek
>             Fix For: 0.7 beta 1
>
>         Attachments: 0001-use-correct-constructor-when-creating-static-CFM-ins.patch
>
>
> This means they are getting assigned ids > 1000.  Static CFMs should be using the private constructor that uses a specific cfid that is <1000
> I'm pretty sure they just need to be getting a readRepairChance=0.

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


[jira] Commented: (CASSANDRA-1354) Static CFMetaData objects are using the wrong constructor.

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

Hudson commented on CASSANDRA-1354:
-----------------------------------

Integrated in Cassandra #509 (See [http://hudson.zones.apache.org/hudson/job/Cassandra/509/])
    use correct constructor when creating static CFM instances. patch by gdusbabek, reviewed by jbellis. CASSANDRA-1354


> Static CFMetaData objects are using the wrong constructor.
> ----------------------------------------------------------
>
>                 Key: CASSANDRA-1354
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-1354
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Gary Dusbabek
>            Assignee: Gary Dusbabek
>             Fix For: 0.7 beta 1
>
>         Attachments: 0001-use-correct-constructor-when-creating-static-CFM-ins.patch
>
>
> This means they are getting assigned ids > 1000.  Static CFMs should be using the private constructor that uses a specific cfid that is <1000
> I'm pretty sure they just need to be getting a readRepairChance=0.

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


[jira] Updated: (CASSANDRA-1354) Static CFMetaData objects are using the wrong constructor.

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

Gary Dusbabek updated CASSANDRA-1354:
-------------------------------------

    Description: 
This means they are getting assigned ids > 1000.  Static CFMs should be using the private constructor that uses a specific cfid that is <1000

I'm pretty sure they just need to be getting a readRepairChance=0.

  was:
This means they are getting assigned ids > 1000.  Static CFMs should be using the private constructor that uses a specific cfid.

I'm pretty sure they just need to be getting a readRepairChance=0.


> Static CFMetaData objects are using the wrong constructor.
> ----------------------------------------------------------
>
>                 Key: CASSANDRA-1354
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-1354
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Gary Dusbabek
>            Assignee: Gary Dusbabek
>             Fix For: 0.7 beta 1
>
>
> This means they are getting assigned ids > 1000.  Static CFMs should be using the private constructor that uses a specific cfid that is <1000
> I'm pretty sure they just need to be getting a readRepairChance=0.

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