Skip to content
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

Enhance performance tool #710

Closed
Closed
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
5ca0c50
add pattern subscribe
harryteng9527 Sep 13, 2022
4b3daa5
tracker support printing sticky partitions
harryteng9527 Sep 14, 2022
cc7d9c3
add sticky partitions to file writer
harryteng9527 Sep 14, 2022
9df8d5f
Merge branch 'main' into enhance-per-tool
harryteng9527 Sep 14, 2022
66f0d4b
spotless
harryteng9527 Sep 14, 2022
ae0db49
Merge 'main' into enhance-per-tool
harryteng9527 Sep 16, 2022
fe48a0f
fix conflit
harryteng9527 Sep 22, 2022
5ea22f2
Merge branch 'main' into enhance-per-tool
harryteng9527 Sep 23, 2022
c3bfc23
delete recording sticky partition of each consumer
harryteng9527 Sep 23, 2022
4f1164d
support printing sticky partition
harryteng9527 Sep 27, 2022
7402d26
Merge branch 'main' into enhance-per-tool
harryteng9527 Sep 27, 2022
a649f2a
fix TrackerTest#testConsumerPrinter
harryteng9527 Sep 27, 2022
6791cef
spotless
harryteng9527 Sep 27, 2022
1b56589
add recordListener to pass client ID & listener
harryteng9527 Sep 28, 2022
edc830b
Merge branch 'main' into enhance-per-tool
harryteng9527 Sep 28, 2022
06df7a9
Merge branch and fix conflict
harryteng9527 Oct 9, 2022
e415c24
delete RecordListener and record partition in consumer
harryteng9527 Oct 11, 2022
ccdabd5
Merge branch 'main' into enhance-per-tool
harryteng9527 Oct 11, 2022
c84cecb
Merge branch 'main' into enhance-per-tool
harryteng9527 Oct 12, 2022
f9d7f08
fix some style and add comments
harryteng9527 Oct 12, 2022
30a84ad
Merge branch 'main' into enhance-per-tool
harryteng9527 Oct 14, 2022
8647028
delete detecting client id if it's null
harryteng9527 Oct 14, 2022
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'main' into enhance-per-tool
harryteng9527 committed Oct 11, 2022

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
commit ccdabd50d91cddd2d427a558fa59f32ecbe55af5
24 changes: 16 additions & 8 deletions app/src/main/java/org/astraea/app/performance/Performance.java
Original file line number Diff line number Diff line change
@@ -98,25 +98,33 @@ public static List<String> execute(final Argument param)
param.consumers,
(clientId, listener) ->
Consumer.forTopics(new HashSet<>(param.topics))
.bootstrapServers(param.bootstrapServers())
.groupId(param.groupId)
.configs(param.configs())
.isolation(param.isolation())
.config(
ConsumerConfigs.ISOLATION_LEVEL_CONFIG,
param.transactionSize > 1
? ConsumerConfigs.ISOLATION_LEVEL_COMMITTED
: ConsumerConfigs.ISOLATION_LEVEL_UNCOMMITTED)
.bootstrapServers(param.bootstrapServers())
.config(ConsumerConfigs.GROUP_ID_CONFIG, param.groupId)
.seek(latestOffsets)
.consumerRebalanceListener(listener)
.clientId(clientId)
.config(ConsumerConfigs.CLIENT_ID_CONFIG, clientId)
.build())
: ConsumerThread.create(
param.consumers,
(clientId, listener) ->
Consumer.forTopics(param.pattern)
.bootstrapServers(param.bootstrapServers())
.groupId(param.groupId)
.configs(param.configs())
.isolation(param.isolation())
.config(
ConsumerConfigs.ISOLATION_LEVEL_CONFIG,
param.transactionSize > 1
? ConsumerConfigs.ISOLATION_LEVEL_COMMITTED
: ConsumerConfigs.ISOLATION_LEVEL_UNCOMMITTED)
.bootstrapServers(param.bootstrapServers())
.config(ConsumerConfigs.GROUP_ID_CONFIG, param.groupId)
.seek(latestOffsets)
.consumerRebalanceListener(listener)
.clientId(clientId)
.config(ConsumerConfigs.CLIENT_ID_CONFIG, clientId)
.build());

System.out.println("creating tracker");
You are viewing a condensed version of this merge commit. You can view the full changes here.