Skip to content

fix: preserve externally provided earcut pool - #219

Merged
kylebarron merged 1 commit into
geoarrow:mainfrom
MarkusTomio:markus/fix-earcut-pool-ownership
Sep 2, 2026
Merged

kylebarron merged 1 commit into
geoarrow:mainfrom
MarkusTomio:markus/fix-earcut-pool-ownership

Conversation

@MarkusTomio

Copy link
Copy Markdown
Contributor

Fixes #218.

This change tracks whether the layer created the pool itself. If it did, it terminates the pool during finalization. If the pool was passed in externally, the layer does not terminate the pool.

A regression test was added to verify that an externally provided pool is not terminated.

@MarkusTomio MarkusTomio changed the title fix: preserve externally provided earcut pool (#218) fix: preserve externally provided earcut pool Aug 26, 2026

@kylebarron kylebarron left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@kylebarron
kylebarron merged commit 3d9215c into geoarrow:main Sep 2, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SolidPolygonLayer should only terminate a worker pool that it created

2 participants