prestodb/presto

ST_Centroid (Java) does not comply with ISO spec

Ouverte

#26 947 ouverte le 12 janv. 2026

Voir sur GitHubGuide contributeur
 (3 commentaires) (0 réaction) (0 personne assignée)Java (5 240 forks)batch import
buggood first issue

Métriques du dépôt

Stars
 (15 558 étoiles)
Métriques de merge PR
 (Merge moyen 34j 14h) (120 PRs mergées en 30 j)

Description

In the ISO spec, ST_Centroid(empty_geometry) returns NULL, but in the Java code it returns an empty geometry. This is fixed for Velox/native in https://github.com/facebookincubator/velox/pull/15985 , but should also be fixed in Java.

Expected Behavior

ST_Centroid(ST_GeometryFromText('LINESTRING EMPTY')) IS NULL

Current Behavior

ST_IsEmpty(ST_Centroid(ST_GeometryFromText('LINESTRING EMPTY')))

Possible Solution

The ST_Centroid function should check for an empty input geometry and return null.

Steps to Reproduce

SELECT ST_Centroid(ST_GeometryFromText('LINESTRING EMPTY'))

Guide contributeur

Direction de recherche
Localisez l'implémentation de ST Centroid dans le code Java, trouvez où les géométries vides sont traitées et ajoutez une condition pour retourner null pour les géométries d'entrée vides.
Stack technique
javasql
Domaine
backendapi
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaSQL
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75