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

Revert "Updates Python ExternalTransform to use the transform service when needed (#27228)" #27343

Closed
wants to merge 1 commit into from

Conversation

bvolpato
Copy link
Contributor

@bvolpato bvolpato commented Jul 3, 2023

This reverts commit d545243 (#27228)

It is currently failing tests with:

ValueError: Cannot start an expansion service since neither Java nor Docker executables are available in the system.

For environments that were working completely fine. This validation was added by #27228

GitHub Actions Tests Status (on master branch)

Build python source distribution and wheels
Python tests
Java tests
Go tests

See CI.md for more information about GitHub Actions CI.

@bvolpato
Copy link
Contributor Author

bvolpato commented Jul 3, 2023

R: @chamikaramj

@github-actions github-actions bot added the python label Jul 3, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jul 3, 2023

Stopping reviewer notifications for this pull request: review requested by someone other than the bot, ceding control

@codecov
Copy link

codecov bot commented Jul 3, 2023

Codecov Report

Merging #27343 (4b758fc) into master (c36f0f1) will decrease coverage by 0.18%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master   #27343      +/-   ##
==========================================
- Coverage   71.49%   71.31%   -0.18%     
==========================================
  Files         858      853       -5     
  Lines      104809   103536    -1273     
==========================================
- Hits        74934    73841    -1093     
+ Misses      28327    28147     -180     
  Partials     1548     1548              
Flag Coverage Δ
python 80.73% <100.00%> (-0.10%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...dks/python/apache_beam/options/pipeline_options.py 94.19% <ø> (-0.02%) ⬇️
sdks/python/apache_beam/transforms/external.py 81.12% <100.00%> (+5.04%) ⬆️

... and 21 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@chamikaramj
Copy link
Contributor

I think the failure is very specific to the way the failing test is run internally at Google (so not a widespread issue). Testing a fix.

@chamikaramj
Copy link
Contributor

Sent #27345

@bvolpato
Copy link
Contributor Author

bvolpato commented Jul 5, 2023

Thanks @chamikaramj, closing this for now.

@bvolpato bvolpato closed this Jul 5, 2023
@bvolpato bvolpato deleted the revert-27228 branch July 5, 2023 04:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants