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

fix: parallel execution and verbose grpc logs #1004

Merged
merged 2 commits into from Sep 13, 2021

Conversation

igorbernstein2
Copy link
Contributor

@igorbernstein2 igorbernstein2 commented Sep 10, 2021

Mucking around with java util logging fails when tests are executed in parallel.
So make the 2 features exclusive: by default tests are executed in parallel, if verbose logs are needed
they can be activated by a profile that will disable parallel execution.

Also switch parallel tests to run in threads instead of processes
and enable parallel execution for unit tests

Fixes #692

Mucking around with java util logging fails when tests are executed in parallel. So make the 2 features exclusive: by default tests are executed in parallel, if verbose logs are needed they can be activated by a profile that will disable parallel execution.

Also switch parallel tests to run in threads instead of processes and enable parallel execution for unit tests
@igorbernstein2 igorbernstein2 requested review from as code owners Sep 10, 2021
@product-auto-label product-auto-label bot added the api: bigtable label Sep 10, 2021
@google-cla google-cla bot added the cla: yes label Sep 10, 2021
@igorbernstein2 igorbernstein2 requested a review from mutianf Sep 10, 2021
@igorbernstein2 igorbernstein2 changed the title chore: fix parallel execution and verbose grpc logs fix: parallel execution and verbose grpc logs Sep 10, 2021
kolea2
kolea2 approved these changes Sep 13, 2021
@igorbernstein2 igorbernstein2 merged commit dd5164d into googleapis:master Sep 13, 2021
15 checks passed
@igorbernstein2 igorbernstein2 deleted the verbose-logs branch Sep 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: bigtable cla: yes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants