-
Notifications
You must be signed in to change notification settings - Fork 252
allow useLabelsAsComponents to be set locally on the ConnectedComponents instance #632
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
allow useLabelsAsComponents to be set locally on the ConnectedComponents instance #632
Conversation
08e1c2f
to
54c9d98
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files@@ Coverage Diff @@
## master #632 +/- ##
==========================================
- Coverage 87.82% 80.74% -7.08%
==========================================
Files 22 24 +2
Lines 1092 1184 +92
Branches 124 148 +24
==========================================
- Hits 959 956 -3
- Misses 133 228 +95 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
a1421b4
to
928c6a6
Compare
@james-willis I think it is my fault, I was going to work on the docs about how to contribute to the spark-connect plugin but as always I failed due to limited time... tldr:
|
Or just add it to the core and I will do all the corresponding updates of PySpark APIs (both classic and connect) |
928c6a6
to
42af82e
Compare
Surely this time is the charm! my local python tests aren't set up so I've been using the CI as a test platform. |
42af82e
to
01de941
Compare
This change lets useLabelsAsComponents to be set locally rather than globally via a spark config. This lets libraries that consume graphframes to avoid leaky abstraction and enables other usecases where affecting the global state of the cluster is inappropriate. See discussion here: apache/sedona#2098
What changes were proposed in this pull request?
Why are the changes needed?