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.

Sign up for GitHub

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

Jump to bottom

[Feature][Api] Refactor org.apache.dolphinscheduler.api.controller.DataAnalysisController #11754

Open
3 tasks done
Zzih opened this issue Sep 2, 2022 · 1 comment
Open
3 tasks done

[Feature][Api] Refactor org.apache.dolphinscheduler.api.controller.DataAnalysisController #11754

Zzih opened this issue Sep 2, 2022 · 1 comment
Assignees
@Zzih
Labels
backend feature new feature

Comments

@Zzih
Copy link
Contributor

Zzih commented Sep 2, 2022 •
edited
Loading

Search before asking

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

Description

#10257 Suggest refactor the backend api
Refactor org.apache.dolphinscheduler.api.controller.TaskInstanceController

Suggested optimization

  • Content-Type change to application/json
  • Add more example to api doc
  • Improve the accuracy of error reporting
  • The input and output parameters of API should use entity

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Code of Conduct
@Zzih Zzih added feature new feature Waiting for reply Waiting for reply labels Sep 2, 2022
@Zzih Zzih changed the title [Feature][Api] Refactor org.apache.dolphinscheduler.api.controller.DataAnalysisV2Controller [Feature][Api] Refactor org.apache.dolphinscheduler.api.controller.DataAnalysisController Sep 2, 2022
@github-actions
Copy link

github-actions bot commented Sep 2, 2022

Thank you for your feedback, we have received your issue, Please wait patiently for a reply.

  • In order for us to understand your request as soon as possible, please provide detailed information、version or pictures.
  • If you haven't received a reply for a long time, you can join our slack and send your question to channel #troubleshooting

@Amy0104 Amy0104 added backend and removed Waiting for reply Waiting for reply labels Sep 2, 2022
This was referenced Sep 5, 2022
Closed
Closed
@Zzih Zzih mentioned this issue Nov 22, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Zzih Zzih

Labels
backend feature new feature
Projects
None yet
Milestone
No milestone
2 participants
@Zzih @Amy0104