[jira] [Commented] (KAFKA-7944) Add more natural Suppress test

2019-03-12 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on KAFKA-7944:
---

guozhangwang commented on pull request #6382: KAFKA-7944: Improve Suppress test 
coverage
URL: https://github.com/apache/kafka/pull/6382
 
 
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Add more natural Suppress test
> --
>
> Key: KAFKA-7944
> URL: https://issues.apache.org/jira/browse/KAFKA-7944
> Project: Kafka
>  Issue Type: Improvement
>  Components: streams
>Reporter: John Roesler
>Assignee: John Roesler
>Priority: Major
>
> Can be integration or system test.
> The idea is to test with tighter time bounds with windows of say 30 seconds 
> and use system time without adding any extra time for verification.
> Currently, all the tests rely on artificially advancing system time, which 
> should be reliable, but you never know. So, we want to add a test that works 
> exactly like production code.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (KAFKA-7944) Add more natural Suppress test

2019-03-06 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on KAFKA-7944:
---

vvcephei commented on pull request #6382: KAFKA-7944: Improve Suppress test 
coverage
URL: https://github.com/apache/kafka/pull/6382
 
 
   * add a normal windowed suppress with short windows and a short grace
 period
   * improve the smoke test so that it actually verifies the intended
 conditions
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Add more natural Suppress test
> --
>
> Key: KAFKA-7944
> URL: https://issues.apache.org/jira/browse/KAFKA-7944
> Project: Kafka
>  Issue Type: Improvement
>  Components: streams
>Reporter: John Roesler
>Priority: Major
>
> Can be integration or system test.
> The idea is to test with tighter time bounds with windows of say 30 seconds 
> and use system time without adding any extra time for verification.
> Currently, all the tests rely on artificially advancing system time, which 
> should be reliable, but you never know. So, we want to add a test that works 
> exactly like production code.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)