-
Notifications
You must be signed in to change notification settings - Fork 13.6k
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-16223: Replace EasyMock/PowerMock with Mockito for KafkaConfigBackingStoreTest #15933
Conversation
a01a64d
to
3fe732d
Compare
@@ -960,6 +666,14 @@ private void expectConvertWriteRead(final String configKey, final Schema valueSc | |||
}); | |||
} | |||
|
|||
private void expectConvertWriteAndRead(final String configKey, final Schema valueSchema, final byte[] serialized, |
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.
please remove this unused function
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.
Removed it. Thanks.
3fe732d
to
42c5b4f
Compare
@FrankYang0529 Could you please rebase code to trigger QA ? |
@chia7712 I triggered a rebuild. You can do it to if you login to the underlying Jenkins: https://issues.apache.org/jira/browse/INFRA-25442 |
Big thanks! |
…ackingStoreTest Signed-off-by: PoAn Yang <payang@apache.org>
42c5b4f
to
34212aa
Compare
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
…ckingStoreTest (apache#15933) Reviewers: Chia-Ping Tsai <chia7712@gmail.com>
Committer Checklist (excluded from commit message)