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?