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

Expose some monitor metrics #106

Merged

Conversation

amehochan
Copy link
Contributor

No description provided.

@amehochan amehochan requested review from nlu90 and a team as code owners December 19, 2022 03:12
@github-actions
Copy link

@amehochan:Thanks for your contribution. For this PR, do we need to update docs?
(The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)

@github-actions github-actions bot added the doc-info-missing This pr needs to mark a document option in description label Dec 19, 2022
@nlu90
Copy link
Collaborator

nlu90 commented Jan 10, 2023

@amehochan Thanks for the contribution.
It seems there are some code lint issues. Could you help resolve it, and then we can merge the PR?

@amehochan amehochan force-pushed the feature/expose-monitor-metrics branch from 8c478c1 to c8bf93d Compare February 8, 2023 14:01
@syhily
Copy link
Contributor

syhily commented Feb 8, 2023

The code lint issues matters. I have passed the checkstyle on my local machine. And format the code by using same formatter like spark.

@amehochan amehochan force-pushed the feature/expose-monitor-metrics branch from c8bf93d to f6e30ef Compare February 8, 2023 14:38
@syhily syhily merged commit d7d955c into streamnative:master Feb 8, 2023
nlu90 added a commit that referenced this pull request Mar 29, 2023
nlu90 added a commit that referenced this pull request Apr 7, 2023
* Revert "Expose some monitor metrics (#106)"

This reverts commit d7d955c.

* Reader need closed when exception (#115)

* fix: Reader did not closed when an exception occurred in readNext

* fix: Reader did not closed when an exception occurred in readNext

* [Improve] replace admin calls with client or consumer calls in PulsarHelper (#118)

* introduce pulsar consumer for pulsar management

* use consumer cache

* lint error

* remove admin url from source related components (#120)

* simplify sink schema check (#121)

---------

Co-authored-by: obobj <vinicolor.violet.end@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-info-missing This pr needs to mark a document option in description
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants