Add latency benchmark #16
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@chinghongfang 我這邊有一些業務需求，因此先做一個版本來測試latency。理想上之後你的#11可以取代這隻暫時的benchmark. |
chia7712
commented
Sep 11, 2021
| import org.apache.kafka.clients.consumer.ConsumerConfig; | ||
| import org.apache.kafka.clients.producer.ProducerConfig; | ||
|
|
||
| interface ComponentFactory { |
There was a problem hiding this comment.
@chinghongfang 麻煩參考看看，這邊示範一個方式如何抽象實作，讓我們方便寫unit test驗證自身邏輯又可以不用真的使用kafka
|
@chia7712 好的，我會盡快修改的，目前正在學習撰寫unit test，會儘快趕上的，讓學長等久了。 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
see https://github.com/chia7712/astraea/tree/latency_benchmark#latency-benchmark for docs