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

Surface errors from Connect workers #3775

Open
ybyzek opened this issue Nov 6, 2019 · 6 comments
Open

Surface errors from Connect workers #3775

ybyzek opened this issue Nov 6, 2019 · 6 comments

Comments

@ybyzek
Copy link
Contributor

ybyzek commented Nov 6, 2019

Is your feature request related to a problem? Please describe.
Users have no visibility into connect worker status and errors in ksqldb

Describe the solution you'd like
Need to surface connect worker (not connector task) status and errors to ksqldb

@agavra
Copy link
Contributor

agavra commented Jan 15, 2020

@ybyzek - I believe this was fixed by #3289 it will show the trace if there is some error message (NOTE that you need to DESCRIBE CONNECTOR to get this information)

@ybyzek
Copy link
Contributor Author

ybyzek commented Jan 15, 2020

@agavra for clarification, will it show the trace if there is an error message for the connect worker or connector? The need is for both.

@agavra
Copy link
Contributor

agavra commented Jan 15, 2020

both 😄 the output will contain the trace for the connector in the top level and the trace for each task in the table below

You can see the test here: https://github.com/confluentinc/ksql/blob/master/ksql-cli/src/test/java/io/confluent/ksql/cli/console/ConsoleTest.java#L633 (I mock the trace for both)

@ybyzek
Copy link
Contributor Author

ybyzek commented Jan 15, 2020

@agavra If no connector has been submitted, but the connect worker has failed, how does it get surfaced? Wouldn't DESCRIBE CONNECTOR be blank?

@agavra
Copy link
Contributor

agavra commented Jan 15, 2020

oh now I understand.. I thought the distinction was between task and connector. You are right, if the connect worker fails this is only shown in the logs

@ybyzek
Copy link
Contributor Author

ybyzek commented Jan 15, 2020

Right, so a few levels to reporting. IIUC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants