-
Notifications
You must be signed in to change notification settings - Fork 256
[query] run ServiceBackend in Py4jQueryDriver
#14984
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
Open
ehigham
wants to merge
1
commit into
ehigham/query-cloud-credentials
Choose a base branch
from
ehigham/experimental-init
base: ehigham/query-cloud-credentials
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
+368
−63
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This was referenced Aug 1, 2025
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
9803dda to
4900338
Compare
970b53e to
491b8dc
Compare
bfb03ff to
969af37
Compare
0fab4fd to
9d9d912
Compare
969af37 to
65ae981
Compare
9d9d912 to
2e55e71
Compare
65ae981 to
b53a9fb
Compare
2e55e71 to
c97beef
Compare
cced9ef to
f4eebb5
Compare
a35e964 to
1b2e3ed
Compare
f4eebb5 to
8f3b9da
Compare
1b2e3ed to
d6df5c0
Compare
8f3b9da to
cd9fa56
Compare
d6df5c0 to
11d12b1
Compare
9ec4390 to
18a198a
Compare
11d12b1 to
a492cc5
Compare
18a198a to
4b0a84a
Compare
a492cc5 to
4c46605
Compare
e2262ff to
7b85699
Compare
73ace30 to
035da72
Compare
52bac9f to
2298818
Compare
035da72 to
7699b2f
Compare
2298818 to
c3f6f77
Compare
7699b2f to
58666e6
Compare
c3f6f77 to
e0b0e08
Compare
58666e6 to
800a403
Compare
e0b0e08 to
6e30672
Compare
800a403 to
a2a0217
Compare
6e30672 to
8b200ea
Compare
a2a0217 to
8b81868
Compare
8b200ea to
effc98f
Compare
9219faf to
6f58b61
Compare
a0050e0 to
64c5584
Compare
64c5584 to
7f247d9
Compare
6f58b61 to
e7b4dea
Compare
d46195e to
44d1bc5
Compare
44d1bc5 to
71be1cd
Compare
e7b4dea to
a0ce020
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.

This PR adds EXPERIMENTAL support for query-on-batch-in-batch by running the driver process locally while offloading partition computation to Batch. This differs from the traditional "batch" backend which requires an additional driver job.
By adding a new experimental feature, my hope is to commit this small proof of concept and then improve it incrementally.
Key changes:
pyServiceBackendmethod to create a ServiceBackend from PythonSecurity Assessment
This change has no impact on the hail batch service in GCP.