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

test: add astra-streaming source test and streamnative-cloud source test #8807

Closed
wants to merge 15 commits into from

Conversation

Honeta
Copy link
Contributor

@Honeta Honeta commented Mar 28, 2023

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.

What's changed and what's your intention?

add two e2e tests, as title

some commits rely on #8428

Checklist For Contributors

  • [ ] I have written necessary rustdoc comments
  • I have added necessary unit tests and integration tests
  • [ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features Sqlsmith: Sql feature generation #7934).
  • [ ] I have demonstrated that backward compatibility is not broken by breaking changes and created issues to track deprecated features to be removed in the future. (Please refer to the issue)
  • All checks passed in ./risedev check (or alias, ./risedev c)

Checklist For Reviewers

  • I have requested macro/micro-benchmarks as this PR can affect performance substantially, and the results are shown.

Documentation

  • My PR DOES NOT contain user-facing changes.

@Honeta Honeta requested a review from tabVersion March 28, 2023 06:12
@github-actions github-actions bot added the component/test Test related issue. label Mar 28, 2023
@Honeta Honeta requested a review from huangjw806 March 28, 2023 06:17
ci/workflows/main-cron.yml Outdated Show resolved Hide resolved
ci/scripts/astra-streaming-source-test.sh Outdated Show resolved Hide resolved
e2e_test/source/pulsar/astra-streaming.py Outdated Show resolved Hide resolved
e2e_test/source/pulsar/streamnative-cloud.py Outdated Show resolved Hide resolved
e2e_test/source/pulsar/astra-streaming.py Outdated Show resolved Hide resolved
e2e_test/source/pulsar/astra-streaming.py Outdated Show resolved Hide resolved
@tabVersion
Copy link
Contributor

the git does not work as you wish, you should merge this pr into #8428 , and then merge it into main. Or merge 8428 first and then impl the tests

@tabVersion
Copy link
Contributor

close for no updates

@tabVersion tabVersion closed this Apr 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/test Test related issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants