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

take bendctl to replace perf-tool #108

Open
flaneur2020 opened this issue Aug 23, 2022 · 1 comment
Open

take bendctl to replace perf-tool #108

flaneur2020 opened this issue Aug 23, 2022 · 1 comment

Comments

@flaneur2020
Copy link
Member

flaneur2020 commented Aug 23, 2022

now we have got the cli to access our cloud platform bendctl, we can consider replacing the cli perf-tool to bendctl.

the problems with perf-tool:

  • the test logic is hard coded inside the cli tool, hard to customize the test logic by python script.
  • the api client is manual written and have some adhoc logic inside, it does not handle some corner cases like 520 well.
  • hard to reproduce the failed case on desktop.

the pros about bendctl:

  • allowing us seperate the api client logic with the test workflow logic (like run each query 10 times for ontime dataset)
  • have a more stable fundation about the api access logic like session management, warehouse management, retry, etc.
@flaneur2020
Copy link
Member Author

#110 replaces the reload part into bendctl.

we can further replace the benchmark part into bendctl.

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

No branches or pull requests

1 participant