Skip to content

Conversation

@james-willis
Copy link
Collaborator

Did you read the Contributor Guide?

Is this PR related to a JIRA ticket?

What changes were proposed in this PR?

  • Rename all the classes related to Geostats SQL to have names related to their functionality - creating custom physical nodes
  • Create Expression Subclasses for both RDD and Dataframe based Physical Functions
  • Merge the UDFRegistrator functionality into the AbstractCatalog so that users subclassing it get bundled functionality for adding their functions to the spark session.

How was this patch tested?

Unit Tests cover these refactors

Did this PR include necessary documentation updates?

  • Yes, I have updated the documentation.

PhysicalFunction and DataFramePhysicalFunction have docstrings to help users of them

@jiayuasu jiayuasu added this to the sedona-1.7.1 milestone Feb 13, 2025
@jiayuasu jiayuasu merged commit 3a3b8d3 into apache:master Feb 13, 2025
40 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants