ulysses-you commented on PR #60078: URL: https://github.com/apache/spark/pull/60078#issuecomment-6653005276
Updated the branch (3900efbe8ee): - Removed the GEOMETRY/GEOGRAPHY rendering branch. A geometry key is a `BinaryView`, and `KeyedPartitioning.apply` fails in `InterpretedHashFunction.hash` with a `MatchError` before any key is built, so the branch was unreachable. The `MapType` branch stays: map keys hash and group end to end. - Reworded the `KeyedPartitioning` scaladoc: when the expressions and the trailing fields alone exceed the budget, the argument list is truncated from the end. - Moved the `JavaSerializer` import above the implicit-conversions comment it was separating. - Refreshed the PR description: the renderer uses the `Literal` constructor and walks struct/array/map by field type, and the test list now names the scalar, CHAR/VARCHAR, complex/UDT and plan-string tests. Tests: catalyst 31 passed with scalastyle clean, and the SPJ/DSv2 suites 504 passed. -- 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]
