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

[Improve][Exception Management] Unified exception management in whole project #2930

Closed
38 tasks done
TyrantLucifer opened this issue Sep 28, 2022 · 16 comments
Closed
38 tasks done

Comments

@TyrantLucifer
Copy link
Member

TyrantLucifer commented Sep 28, 2022

Search before asking

  • I had searched in the feature and found no similar feature requirement.

Description

So far, the exceptions thrown in the code have not been managed uniformly, the connector and other modules still use RunException, unified exception management can make the code more clear and easy to read, and quickly prompt the user where the exception occurred.

The subtasks to be done are as follows:

Usage Scenario

No response

Related issues

No response

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@TaoZex
Copy link
Contributor

TaoZex commented Nov 22, 2022

I want to work for kafka,Google Sheets,http,feishu,lemlist,myhours,klaviyo,onesignal,starrocks,tablestore,redis and mongodb connector to unified exception.

@TyrantLucifer
Copy link
Member Author

TyrantLucifer commented Nov 23, 2022

I want to work for kafka,Google Sheets,http,feishu,lemlist,myhours,klaviyo,onesignal,starrocks,tablestore,redis and mongodb connector to unified exception.

Please take one task at a time. Assign kafka to you. The owner of connectors do this work is better.

@TaoZex
Copy link
Contributor

TaoZex commented Nov 23, 2022

I want to work for kafka,Google Sheets,http,feishu,lemlist,myhours,klaviyo,onesignal,starrocks,tablestore,redis and mongodb connector to unified exception.

Please take one task at a time. Assign kafka to you. The owner of connectors do this work is better.

Thanks.I will finish the connectors that I am familiar with.

@MonsterChenzhuo
Copy link
Contributor

Please assign wechat connector to me.Thank

@TyrantLucifer
Copy link
Member Author

Please assign wechat connector to me.Thank

#3562

@FWLamb
Copy link
Contributor

FWLamb commented Nov 24, 2022

please assign ElasticSearch source & sink connector to me, thx.

@Zzih
Copy link

Zzih commented Nov 24, 2022

Are there any remaining tasks assigned to me?

@TaoZex
Copy link
Contributor

TaoZex commented Nov 24, 2022

I have unified the exception of formats module, please link the issue:#3571

@cason0126
Copy link
Contributor

I am doing expose configurable options in Hudi 。
Please assign Hudi connector to me.Thanks

@github-actions
Copy link

github-actions bot commented Jan 8, 2023

This issue has been automatically marked as stale because it has not had recent activity for 30 days. It will be closed in next 7 days if no further activity occurs.

@github-actions github-actions bot added the stale label Jan 8, 2023
@TyrantLucifer
Copy link
Member Author

The task was successfully accomplished! Thank you for all contributors.

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

No branches or pull requests