Tao Jiuming comments

Results 61 comments of


                                            Tao Jiuming

[pulsar][receiver] add apache pulsar receiver

@dmitryax All of CI checks passed, PTAL

[pulsar][receiver] add apache pulsar receiver

@dmitryax All CI checks passed, PTAL

[pulsar][receiver] add apache pulsar receiver

@dmitryax never mind. the conflicts has resolved, PTAL

[enh] Make maxAccumulations configurable and add remove method for meters

I'll fix the CI failures later if the PR is acceptable.

[improve][broker] Add gzip compression before http service response

> Having gzip support for all http endpoints could be useful too. I wonder if it could be simply enabled for all by default. @codelipenghui wdyt? I remember this PR...

[improve][broker] Add gzip compression before http service response

for the tests, I remember the HTTP response code is always 302, so they didn't passed, does it fixed? and it's better to make sure the response body could parse...

[improve][broker] Add gzip compression before http service response

besides, over all LGTM

[monitor][txn] Add metrics for transaction

Tests to be completed

[monitor][txn] Add metrics for transaction

> Good work! as the seem as tb, we should add tp stats I know TC means `TransactionCoordinator` and TB means `TransactionBuffer`, but what means TP?

[monitor][txn] Add metrics for transaction

> > Good work! as the seem as tb, we should add tp stats > > I know TC means `TransactionCoordinator` and TB means `TransactionBuffer`, but what means TP? Does...