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

Feature(e2e_test): new api of e2e executor; add case 'data lost after compaction and restart' #2120

Merged
merged 2 commits into from
May 21, 2024

Conversation

Benxiaohai001
Copy link
Member

@Benxiaohai001 Benxiaohai001 commented May 13, 2024

Required checklist

  • Sample config files updated (config,meta/config and default config)
  • If there are user-facing changes, the documentation needs to be updated prior to approving the PR( Link )
  • If there are any breaking changes to public APIs, please add the api change label.
  • Signed CLA (if not already signed)

Which issue does this PR close?

Related #.

Rationale for this change

  • Make old API of e2e test case executor deprecated, move Step to StepLegacy, execute_steps() to execute_steps_legacy().
  • Add new API for e2e test case executor, add trait Step, struct E2eContext, add step ShellStep to execute shell commands.
    • Some implementations of Step have fields that can lazy init their values using a reference of E2eContext, or can even modify context variables in E2eContext before or after their execution.
    • CnosdbMetaTestHelper and CnosdbDataTestHelper now store configs of cnosdb meta nodes (meta::store::config::Opt) and data nodes (config::Config).
  • Add restart test case_6 to test whether cnosdb will lost data after a compaction and restart.

Are there any user-facing changes?

@zipper-meng zipper-meng changed the title [testcase]Add test case about "select * " Feature(e2e_test): new api of e2e executor; add case 'data lost after compaction and restart' May 13, 2024
@roseboy-liu roseboy-liu merged commit 04c20c7 into LTS/2.3 May 21, 2024
7 checks passed
@roseboy-liu roseboy-liu deleted the e2e_test/compact_restart branch May 21, 2024 06:02
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

Successfully merging this pull request may close these issues.

None yet

3 participants