You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "T Jake Luciani (JIRA)" <ji...@apache.org> on 2011/01/24 04:40:43 UTC

[jira] Created: (CASSANDRA-2038) Add local read path back to StorageProxy

Add local read path back to StorageProxy
----------------------------------------

                 Key: CASSANDRA-2038
                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2038
             Project: Cassandra
          Issue Type: Improvement
          Components: Core
    Affects Versions: 0.7.1
            Reporter: T Jake Luciani
            Assignee: T Jake Luciani
             Fix For: 0.7.1



CASSANDRA-982 removed the local read path optimization.  Without this all local reads go through serialization and messaging.  Avoiding this step improves local read performance > 50%

 

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


[jira] Commented: (CASSANDRA-2038) Add local read path back to StorageProxy

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

Hudson commented on CASSANDRA-2038:
-----------------------------------

Integrated in Cassandra-0.7 #197 (See [https://hudson.apache.org/hudson/job/Cassandra-0.7/197/])
    add optimization for local reads to StorageProxy
patch by jbellis and tjake for CASSANDRA-2038


> Add local read path back to StorageProxy
> ----------------------------------------
>
>                 Key: CASSANDRA-2038
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2038
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>    Affects Versions: 0.7.1
>            Reporter: T Jake Luciani
>            Assignee: Jonathan Ellis
>             Fix For: 0.7.1
>
>         Attachments: 0001-CASSANDRA-2038-reincorporate-local-reads.txt, 0002-CASSANDRA-2038-only-local-for-CL.ONE.txt, 2038-v2.txt
>
>   Original Estimate: 6h
>  Remaining Estimate: 6h
>
> CASSANDRA-982 removed the local read path optimization.  Without this all local reads go through serialization and messaging.  Avoiding this step improves local read performance > 50%
>  

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


[jira] Updated: (CASSANDRA-2038) Add local read path back to StorageProxy

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

T Jake Luciani updated CASSANDRA-2038:
--------------------------------------

    Attachment: 0001-CASSANDRA-2038-reincorporate-local-reads.txt

> Add local read path back to StorageProxy
> ----------------------------------------
>
>                 Key: CASSANDRA-2038
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2038
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>    Affects Versions: 0.7.1
>            Reporter: T Jake Luciani
>            Assignee: T Jake Luciani
>             Fix For: 0.7.1
>
>         Attachments: 0001-CASSANDRA-2038-reincorporate-local-reads.txt
>
>
> CASSANDRA-982 removed the local read path optimization.  Without this all local reads go through serialization and messaging.  Avoiding this step improves local read performance > 50%
>  

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


[jira] Updated: (CASSANDRA-2038) Add local read path back to StorageProxy

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

T Jake Luciani updated CASSANDRA-2038:
--------------------------------------

    Attachment: 0002-CASSANDRA-2038-only-local-for-CL.ONE.txt
                0001-CASSANDRA-2038-reincorporate-local-reads.txt

> Add local read path back to StorageProxy
> ----------------------------------------
>
>                 Key: CASSANDRA-2038
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2038
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>    Affects Versions: 0.7.1
>            Reporter: T Jake Luciani
>            Assignee: T Jake Luciani
>             Fix For: 0.7.1
>
>         Attachments: 0001-CASSANDRA-2038-reincorporate-local-reads.txt, 0002-CASSANDRA-2038-only-local-for-CL.ONE.txt
>
>
> CASSANDRA-982 removed the local read path optimization.  Without this all local reads go through serialization and messaging.  Avoiding this step improves local read performance > 50%
>  

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


[jira] Commented: (CASSANDRA-2038) Add local read path back to StorageProxy

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

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

I tested this with stress on a 4 node cluster with RF=2, CL.ONE reads and CL.ALL reads no issues.

> Add local read path back to StorageProxy
> ----------------------------------------
>
>                 Key: CASSANDRA-2038
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2038
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>    Affects Versions: 0.7.1
>            Reporter: T Jake Luciani
>            Assignee: T Jake Luciani
>             Fix For: 0.7.1
>
>         Attachments: 0001-CASSANDRA-2038-reincorporate-local-reads.txt, 0002-CASSANDRA-2038-only-local-for-CL.ONE.txt, 2038-v2.txt
>
>
> CASSANDRA-982 removed the local read path optimization.  Without this all local reads go through serialization and messaging.  Avoiding this step improves local read performance > 50%
>  

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


[jira] Resolved: (CASSANDRA-2038) Add local read path back to StorageProxy

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

Jonathan Ellis resolved CASSANDRA-2038.
---------------------------------------

    Resolution: Fixed
      Reviewer: tjake
      Assignee: Jonathan Ellis  (was: T Jake Luciani)

committed

> Add local read path back to StorageProxy
> ----------------------------------------
>
>                 Key: CASSANDRA-2038
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2038
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>    Affects Versions: 0.7.1
>            Reporter: T Jake Luciani
>            Assignee: Jonathan Ellis
>             Fix For: 0.7.1
>
>         Attachments: 0001-CASSANDRA-2038-reincorporate-local-reads.txt, 0002-CASSANDRA-2038-only-local-for-CL.ONE.txt, 2038-v2.txt
>
>   Original Estimate: 6h
>  Remaining Estimate: 6h
>
> CASSANDRA-982 removed the local read path optimization.  Without this all local reads go through serialization and messaging.  Avoiding this step improves local read performance > 50%
>  

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


[jira] Updated: (CASSANDRA-2038) Add local read path back to StorageProxy

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

Jonathan Ellis updated CASSANDRA-2038:
--------------------------------------

           Description: 
CASSANDRA-982 removed the local read path optimization.  Without this all local reads go through serialization and messaging.  Avoiding this step improves local read performance > 50%

 

  was:

CASSANDRA-982 removed the local read path optimization.  Without this all local reads go through serialization and messaging.  Avoiding this step improves local read performance > 50%

 

    Remaining Estimate: 6h
     Original Estimate: 6h

> Add local read path back to StorageProxy
> ----------------------------------------
>
>                 Key: CASSANDRA-2038
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2038
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>    Affects Versions: 0.7.1
>            Reporter: T Jake Luciani
>            Assignee: T Jake Luciani
>             Fix For: 0.7.1
>
>         Attachments: 0001-CASSANDRA-2038-reincorporate-local-reads.txt, 0002-CASSANDRA-2038-only-local-for-CL.ONE.txt, 2038-v2.txt
>
>   Original Estimate: 6h
>  Remaining Estimate: 6h
>
> CASSANDRA-982 removed the local read path optimization.  Without this all local reads go through serialization and messaging.  Avoiding this step improves local read performance > 50%
>  

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


[jira] Updated: (CASSANDRA-2038) Add local read path back to StorageProxy

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

Jonathan Ellis updated CASSANDRA-2038:
--------------------------------------

    Attachment: 2038-v2.txt

v2 generalizes to allow optimizing local reads for any CL and for digest as well as data reads, without having to special-case LocalFutures.  it also simplifies the message-sending to a single pass instead of message array construction followed by re-iteration over them.

v2 is completely un-tested but the approach is promising. :)

> Add local read path back to StorageProxy
> ----------------------------------------
>
>                 Key: CASSANDRA-2038
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2038
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>    Affects Versions: 0.7.1
>            Reporter: T Jake Luciani
>            Assignee: T Jake Luciani
>             Fix For: 0.7.1
>
>         Attachments: 0001-CASSANDRA-2038-reincorporate-local-reads.txt, 0002-CASSANDRA-2038-only-local-for-CL.ONE.txt, 2038-v2.txt
>
>
> CASSANDRA-982 removed the local read path optimization.  Without this all local reads go through serialization and messaging.  Avoiding this step improves local read performance > 50%
>  

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


[jira] Issue Comment Edited: (CASSANDRA-2038) Add local read path back to StorageProxy

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

T Jake Luciani edited comment on CASSANDRA-2038 at 1/24/11 12:02 PM:
---------------------------------------------------------------------

I tested this with stress on a 4 node cluster with RF=2, CL.ONE reads and CL.ALL reads no issues. +1

      was (Author: tjake):
    I tested this with stress on a 4 node cluster with RF=2, CL.ONE reads and CL.ALL reads no issues.
  
> Add local read path back to StorageProxy
> ----------------------------------------
>
>                 Key: CASSANDRA-2038
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2038
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>    Affects Versions: 0.7.1
>            Reporter: T Jake Luciani
>            Assignee: T Jake Luciani
>             Fix For: 0.7.1
>
>         Attachments: 0001-CASSANDRA-2038-reincorporate-local-reads.txt, 0002-CASSANDRA-2038-only-local-for-CL.ONE.txt, 2038-v2.txt
>
>
> CASSANDRA-982 removed the local read path optimization.  Without this all local reads go through serialization and messaging.  Avoiding this step improves local read performance > 50%
>  

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


[jira] Updated: (CASSANDRA-2038) Add local read path back to StorageProxy

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

T Jake Luciani updated CASSANDRA-2038:
--------------------------------------

    Attachment:     (was: 0001-CASSANDRA-2038-reincorporate-local-reads.txt)

> Add local read path back to StorageProxy
> ----------------------------------------
>
>                 Key: CASSANDRA-2038
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2038
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>    Affects Versions: 0.7.1
>            Reporter: T Jake Luciani
>            Assignee: T Jake Luciani
>             Fix For: 0.7.1
>
>
> CASSANDRA-982 removed the local read path optimization.  Without this all local reads go through serialization and messaging.  Avoiding this step improves local read performance > 50%
>  

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