Skip to content

feat: Add ST_IsEmpty#76

Merged
kylebarron merged 1 commit into
datafusion-contrib:mainfrom
ianthetechie:add-st-is-empty
May 27, 2026
Merged

feat: Add ST_IsEmpty#76
kylebarron merged 1 commit into
datafusion-contrib:mainfrom
ianthetechie:add-st-is-empty

Conversation

@ianthetechie

Copy link
Copy Markdown
Contributor

What it says on the tin :) Seemed like some low-hanging fruit.

@github-actions github-actions Bot added the feat label May 26, 2026

@kylebarron kylebarron left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Can you add to register:

pub fn register(session_context: &datafusion::prelude::SessionContext) {
session_context.register_udf(CoordDim.into());
session_context.register_udf(NDims.into());
session_context.register_udf(GeometryType.into());
session_context.register_udf(ST_GeometryType.into());
session_context.register_udf(IsClosed.into());
session_context.register_udf(EndPoint::default().into());
session_context.register_udf(StartPoint::default().into());
session_context.register_udf(NPoints.into());
session_context.register_udf(NumInteriorRings.into());
session_context.register_udf(M::default().into());
session_context.register_udf(X::default().into());
session_context.register_udf(Y::default().into());
session_context.register_udf(Z::default().into());
}

@ianthetechie

Copy link
Copy Markdown
Contributor Author

I thought I did... but it's also equally probable that I'm a bit dense today, or GitHub PR displays are on the fritz 😅

image

@kylebarron kylebarron left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh I just noticed that file now; thanks!

@kylebarron kylebarron merged commit df0f40a into datafusion-contrib:main May 27, 2026
7 checks passed
@ianthetechie ianthetechie deleted the add-st-is-empty branch May 27, 2026 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants