zhengruifeng commented on PR #58302: URL: https://github.com/apache/spark/pull/58302#issuecomment-5448325714
Test-coverage update for `0cc556dfd3c`: - Ported all 31 upstream GraphFrames Scala test suites. - Ported all 46 core Python tests and all 14 PropertyGraphFrame tests to classic PySpark. - Spark Connect runs 41 applicable core tests, all 14 PropertyGraphFrame tests, plus Connect/client-specific tests. The remaining five upstream core cases depend on classic-only internals. - Copied all eight LDBC tests. They are registered but ignored by default because they require external tools and network datasets; `SPARK_RUN_LDBC_GRAPH_TESTS=1` opts in. - Did not duplicate the external repository's 109 private GraphX-fork tests: canonical Spark already contains their GraphX test coverage. - Validated direct Pregel construction in classic and Connect, including thin-client source-distribution imports. `graphframes/Test/compile`, the 11 newly ported non-LDBC Scala tests, all eight Python module suites (`pyspark-graphframes,pyspark-graphframes-connect`), `assembly/package`, Ruff, targeted Scalafmt, license checks, workflow YAML parsing, and `sparktestsupport` doctests passed locally. There is separate pre-existing draft debt of 208 module-wide Scalastyle violations (108 main, 100 test); this commit adds none of them. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
