New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
KAFKA-7513: Fix timing issue in SaslAuthenticatorFailureDelayTest #5805
KAFKA-7513: Fix timing issue in SaslAuthenticatorFailureDelayTest #5805
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just a minor comment regarding the existing code. No need for re-review.
@@ -53,7 +53,7 @@ | |||
@RunWith(value = Parameterized.class) | |||
public class SaslAuthenticatorFailureDelayTest { | |||
private static final int BUFFER_SIZE = 4 * 1024; | |||
private static MockTime time = new MockTime(50); | |||
private static MockTime time = new MockTime(10); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any reason why this isn't final?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Made it final and changed to instance variable like we have in other tests.
Are any of the failures related? |
@ijuma Thanks for the review. The test failures don't look related. |
@ijuma Thanks for the review. Test failures not related, merging to trunk and 2.1. |
) Reduce tick interval of the mock timer and avoid large timer increments to avoid hitting idle expiry on the client-side before delayed close is processed by the server. Also reduce poll interval in the server to make the test complete faster (since delayed close is only processed when poll returns). Reviewers: Ismael Juma <ismael@juma.me.uk>
…ache#5805) Reduce tick interval of the mock timer and avoid large timer increments to avoid hitting idle expiry on the client-side before delayed close is processed by the server. Also reduce poll interval in the server to make the test complete faster (since delayed close is only processed when poll returns). Reviewers: Ismael Juma <ismael@juma.me.uk>
Reduce tick interval of the mock timer and avoid large timer increments to avoid hitting idle expiry on the client-side before delayed close is processed by the server. Also reduce poll interval in the server to make the test complete faster (since delayed close is only processed when poll returns).
Committer Checklist (excluded from commit message)