[jira] [Commented] (KAFKA-1965) Leaner DelayedItem

2015-02-20 Thread Yasuhiro Matsuda (JIRA)

[ 
https://issues.apache.org/jira/browse/KAFKA-1965?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14329725#comment-14329725
 ] 

Yasuhiro Matsuda commented on KAFKA-1965:
-

Updated reviewboard https://reviews.apache.org/r/31199/diff/
 against branch origin/trunk

> Leaner DelayedItem
> --
>
> Key: KAFKA-1965
> URL: https://issues.apache.org/jira/browse/KAFKA-1965
> Project: Kafka
>  Issue Type: Improvement
>  Components: purgatory
>Reporter: Yasuhiro Matsuda
>Assignee: Joel Koshy
>Priority: Trivial
> Attachments: KAFKA-1965.patch, KAFKA-1965.patch, 
> KAFKA-1965_2015-02-20_15:08:26.patch
>
>
> In DelayedItem, which is a superclass of DelayedOperation, both the creation 
> timestamp and the length delay are stored. However, all it needs is one 
> timestamp that is the due of the item.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (KAFKA-1965) Leaner DelayedItem

2015-02-19 Thread Yasuhiro Matsuda (JIRA)

[ 
https://issues.apache.org/jira/browse/KAFKA-1965?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14327795#comment-14327795
 ] 

Yasuhiro Matsuda commented on KAFKA-1965:
-

Created reviewboard https://reviews.apache.org/r/31199/diff/
 against branch origin/trunk

> Leaner DelayedItem
> --
>
> Key: KAFKA-1965
> URL: https://issues.apache.org/jira/browse/KAFKA-1965
> Project: Kafka
>  Issue Type: Improvement
>  Components: purgatory
>Reporter: Yasuhiro Matsuda
>Assignee: Joel Koshy
>Priority: Trivial
> Attachments: KAFKA-1965.patch, KAFKA-1965.patch
>
>
> In DelayedItem, which is a superclass of DelayedOperation, both the creation 
> timestamp and the length delay are stored. However, all it needs is one 
> timestamp that is the due of the item.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (KAFKA-1965) Leaner DelayedItem

2015-02-18 Thread Sriharsha Chintalapani (JIRA)

[ 
https://issues.apache.org/jira/browse/KAFKA-1965?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14326973#comment-14326973
 ] 

Sriharsha Chintalapani commented on KAFKA-1965:
---

[~yasuhiro.matsuda] Changes looks good to me. Can you attach it to the review 
board here are instructions 
https://cwiki.apache.org/confluence/display/KAFKA/Patch+submission+and+review

> Leaner DelayedItem
> --
>
> Key: KAFKA-1965
> URL: https://issues.apache.org/jira/browse/KAFKA-1965
> Project: Kafka
>  Issue Type: Improvement
>  Components: purgatory
>Reporter: Yasuhiro Matsuda
>Assignee: Joel Koshy
>Priority: Trivial
> Attachments: KAFKA-1965.patch
>
>
> In DelayedItem, which is a superclass of DelayedOperation, both the creation 
> timestamp and the length delay are stored. However, all it needs is one 
> timestamp that is the due of the item.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)