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

Sriharsha Chintalapani commented on STORM-1988:
-----------------------------------------------

[~ptgoetz] This sounds like a mutually exclusive argument in the same by-laws 
page

{code}
The code can be committed after the first +1
{code}
vs
{code}
1 day from initial patch (Note: Committers should consider allowing more time 
for review based on the complexity and/or impact of the patch in question.)
{code}

Either way waiting for 1 day for small patches doesn't make sense and also for 
critical patches. We did indeed merge some of the patches for releases.
I guess thats a discussion for mailing list. I'll wait for 24-hr period until 
we get more clarity on this policy.

> Kafka Offset not showing due to bad classpath
> ---------------------------------------------
>
>                 Key: STORM-1988
>                 URL: https://issues.apache.org/jira/browse/STORM-1988
>             Project: Apache Storm
>          Issue Type: Bug
>          Components: storm-ui
>    Affects Versions: 2.0.0, 1.1.0
>            Reporter: Jungtaek Lim
>            Assignee: Jungtaek Lim
>            Priority: Critical
>
> STORM-1950 breaks classpath of storm-kafka-monitor. 
> Classpath doesn't work with wildcard and filename prefix/postfix. It was 
> added for purposing to prevent other libs to also included as classpath, but 
> it just doesn't work. My bad.
> We should fix classpath to specify full filename path or directory/* pattern.



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

Reply via email to