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

Split pulsar-client-and-admin-shade-test into two parts #8716

Merged
merged 8 commits into from
Dec 1, 2020

Conversation

reswqa
Copy link
Member

@reswqa reswqa commented Nov 26, 2020

Motivation
Add test for #8697 and other shade problems.

Modifications
splitting pulsar-client-and-admin-shade-test into two parts, before the pulsar-admin and pulsar-client shade tests were together, can cause some potential problems.

@reswqa
Copy link
Member Author

reswqa commented Nov 26, 2020

/pulsarbot run-failure-checks

1 similar comment
@jiazhai
Copy link
Member

jiazhai commented Dec 1, 2020

/pulsarbot run-failure-checks

@jiazhai jiazhai merged commit 7f62bbc into apache:master Dec 1, 2020
RobertIndie pushed a commit to RobertIndie/pulsar that referenced this pull request Dec 8, 2020
Motivation
Add test for apache#8697 and other shade problems.

Modifications
splitting pulsar-client-and-admin-shade-test into two parts, before the pulsar-admin and pulsar-client shade tests were together, can cause some potential problems.

* Add CI test for shade package

* fix style

* fix license

* separate shade package

* fix pom.xml

* fix codestyle

* remove unused import

Co-authored-by: WeijieGuo <reswqa@bupt.edu.cn>
@zymap zymap added this to the 2.8.0 milestone Feb 19, 2021
@codelipenghui codelipenghui added the cherry-picked/branch-2.7 Archived: 2.7 is end of life label Feb 19, 2021
codelipenghui pushed a commit that referenced this pull request Feb 19, 2021
Motivation
Add test for #8697 and other shade problems.

Modifications
splitting pulsar-client-and-admin-shade-test into two parts, before the pulsar-admin and pulsar-client shade tests were together, can cause some potential problems.

* Add CI test for shade package

* fix style

* fix license

* separate shade package

* fix pom.xml

* fix codestyle

* remove unused import

Co-authored-by: WeijieGuo <reswqa@bupt.edu.cn>
(cherry picked from commit 7f62bbc)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants