You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Nate McCall (JIRA)" <ji...@apache.org> on 2011/01/25 17:25:45 UTC

[jira] Created: (CASSANDRA-2050) AbstractDaemon unnecessarily uses jetty inteface

AbstractDaemon unnecessarily uses jetty inteface
------------------------------------------------

                 Key: CASSANDRA-2050
                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2050
             Project: Cassandra
          Issue Type: Bug
    Affects Versions: 0.7.1, 0.7.2, 0.8
            Reporter: Nate McCall
            Assignee: Nate McCall


AbstractDaemon's CleaningThreadPool need not implement this jetty interface. Removing this would allow us to remove jetty dependency altogether. 

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


[jira] Commented: (CASSANDRA-2050) AbstractDaemon unnecessarily uses jetty interface

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

Hudson commented on CASSANDRA-2050:
-----------------------------------

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

> AbstractDaemon unnecessarily uses jetty interface
> -------------------------------------------------
>
>                 Key: CASSANDRA-2050
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2050
>             Project: Cassandra
>          Issue Type: Task
>          Components: API
>            Reporter: Nate McCall
>            Assignee: Nate McCall
>            Priority: Minor
>             Fix For: 0.8
>
>         Attachments: 2050.txt
>
>
> AbstractDaemon's CleaningThreadPool need not implement this jetty interface. Removing this would allow us to remove jetty dependency altogether. 

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


[jira] Commented: (CASSANDRA-2050) AbstractDaemon unnecessarily uses jetty interface

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

Stu Hood commented on CASSANDRA-2050:
-------------------------------------

> I think this was only needed for avro plumbing.
Yea: this can be blamed back to when we wanted to better support auth in Avro. Should be safe to remove.

The patch should probably also remove the dependency, right?

> AbstractDaemon unnecessarily uses jetty interface
> -------------------------------------------------
>
>                 Key: CASSANDRA-2050
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2050
>             Project: Cassandra
>          Issue Type: Task
>          Components: API
>            Reporter: Nate McCall
>            Assignee: Nate McCall
>            Priority: Minor
>             Fix For: 0.7.1
>
>         Attachments: 2050.txt
>
>
> AbstractDaemon's CleaningThreadPool need not implement this jetty interface. Removing this would allow us to remove jetty dependency altogether. 

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


[jira] Updated: (CASSANDRA-2050) AbstractDaemon unnecessarily uses jetty interface

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

Jonathan Ellis updated CASSANDRA-2050:
--------------------------------------

             Reviewer: urandom
          Component/s: API
    Affects Version/s:     (was: 0.7.2)
                           (was: 0.7.1)
                           (was: 0.8)
        Fix Version/s: 0.7.1
           Issue Type: Task  (was: Bug)

> AbstractDaemon unnecessarily uses jetty interface
> -------------------------------------------------
>
>                 Key: CASSANDRA-2050
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2050
>             Project: Cassandra
>          Issue Type: Task
>          Components: API
>            Reporter: Nate McCall
>            Assignee: Nate McCall
>            Priority: Minor
>             Fix For: 0.7.1
>
>         Attachments: 2050.txt
>
>
> AbstractDaemon's CleaningThreadPool need not implement this jetty interface. Removing this would allow us to remove jetty dependency altogether. 

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


[jira] Updated: (CASSANDRA-2050) AbstractDaemon unnecessarily uses jetty interface

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

Jonathan Ellis updated CASSANDRA-2050:
--------------------------------------

    Fix Version/s:     (was: 0.7.2)
                   0.8

this breaks stuff in 0.7, moved to 0.8

> AbstractDaemon unnecessarily uses jetty interface
> -------------------------------------------------
>
>                 Key: CASSANDRA-2050
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2050
>             Project: Cassandra
>          Issue Type: Task
>          Components: API
>            Reporter: Nate McCall
>            Assignee: Nate McCall
>            Priority: Minor
>             Fix For: 0.8
>
>         Attachments: 2050.txt
>
>
> AbstractDaemon's CleaningThreadPool need not implement this jetty interface. Removing this would allow us to remove jetty dependency altogether. 

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


[jira] Commented: (CASSANDRA-2050) AbstractDaemon unnecessarily uses jetty interface

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

Hudson commented on CASSANDRA-2050:
-----------------------------------

Integrated in Cassandra #695 (See [https://hudson.apache.org/hudson/job/Cassandra/695/])
    r/m jetty
patch by Nate McCall; reviewed by stuhood for CASSANDRA-2050


> AbstractDaemon unnecessarily uses jetty interface
> -------------------------------------------------
>
>                 Key: CASSANDRA-2050
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2050
>             Project: Cassandra
>          Issue Type: Task
>          Components: API
>            Reporter: Nate McCall
>            Assignee: Nate McCall
>            Priority: Minor
>             Fix For: 0.8
>
>         Attachments: 2050.txt
>
>
> AbstractDaemon's CleaningThreadPool need not implement this jetty interface. Removing this would allow us to remove jetty dependency altogether. 

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


[jira] Updated: (CASSANDRA-2050) AbstractDaemon unnecessarily uses jetty interface

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

Nate McCall updated CASSANDRA-2050:
-----------------------------------

    Attachment: 2050.txt

Remove jetty import and interface impl on AbstractDaemon.

I think this was only needed for avro plumbing. 

> AbstractDaemon unnecessarily uses jetty interface
> -------------------------------------------------
>
>                 Key: CASSANDRA-2050
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2050
>             Project: Cassandra
>          Issue Type: Bug
>    Affects Versions: 0.7.1, 0.7.2, 0.8
>            Reporter: Nate McCall
>            Assignee: Nate McCall
>         Attachments: 2050.txt
>
>
> AbstractDaemon's CleaningThreadPool need not implement this jetty interface. Removing this would allow us to remove jetty dependency altogether. 

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


[jira] Updated: (CASSANDRA-2050) AbstractDaemon unnecessarily uses jetty interface

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

Nate McCall updated CASSANDRA-2050:
-----------------------------------

    Summary: AbstractDaemon unnecessarily uses jetty interface  (was: AbstractDaemon unnecessarily uses jetty inteface)

> AbstractDaemon unnecessarily uses jetty interface
> -------------------------------------------------
>
>                 Key: CASSANDRA-2050
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2050
>             Project: Cassandra
>          Issue Type: Bug
>    Affects Versions: 0.7.1, 0.7.2, 0.8
>            Reporter: Nate McCall
>            Assignee: Nate McCall
>
> AbstractDaemon's CleaningThreadPool need not implement this jetty interface. Removing this would allow us to remove jetty dependency altogether. 

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


[jira] Resolved: (CASSANDRA-2050) AbstractDaemon unnecessarily uses jetty interface

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

Jonathan Ellis resolved CASSANDRA-2050.
---------------------------------------

    Resolution: Fixed
      Reviewer: stuhood  (was: urandom)

committed w/ removal of lib/jetty* (which svn diff won't include in a patch)

> AbstractDaemon unnecessarily uses jetty interface
> -------------------------------------------------
>
>                 Key: CASSANDRA-2050
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2050
>             Project: Cassandra
>          Issue Type: Task
>          Components: API
>            Reporter: Nate McCall
>            Assignee: Nate McCall
>            Priority: Minor
>             Fix For: 0.7.2
>
>         Attachments: 2050.txt
>
>
> AbstractDaemon's CleaningThreadPool need not implement this jetty interface. Removing this would allow us to remove jetty dependency altogether. 

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


[jira] Updated: (CASSANDRA-2050) AbstractDaemon unnecessarily uses jetty interface

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

Nate McCall updated CASSANDRA-2050:
-----------------------------------

    Priority: Minor  (was: Major)

> AbstractDaemon unnecessarily uses jetty interface
> -------------------------------------------------
>
>                 Key: CASSANDRA-2050
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2050
>             Project: Cassandra
>          Issue Type: Bug
>    Affects Versions: 0.7.1, 0.7.2, 0.8
>            Reporter: Nate McCall
>            Assignee: Nate McCall
>            Priority: Minor
>         Attachments: 2050.txt
>
>
> AbstractDaemon's CleaningThreadPool need not implement this jetty interface. Removing this would allow us to remove jetty dependency altogether. 

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