4paradigm / 4paradigm/OpenMLDB

Add const constraints for udf input pointers

Abierto
#2,351 1 comentario 0 reacciones 1 asignado Reclamado por @hezean Ver en GitHub
enhancement execute-engine udf
Lenguaje dominante
C++
Estrellas
1.7k
Forks
331
Merge medio
12 d 12 h
PR fusionados (30 d)
1

Descripción

For other udfs' signature, it would be better you can create a new PR and do the fix, if you are interested. Thanks.

_Originally posted by @zhanghaohit in https://github.com/4paradigm/OpenMLDB/pull/2262#discussion_r944313023_

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Revisa las firmas de UDF en la base de código, especialmente las modificadas en PR #2262. Identifica los parámetros de puntero de entrada que deberían marcarse como const por seguridad. Revisa las implementaciones de UDF existentes para entender el patrón y aplica restricciones const donde corresponda. Las pruebas deben garantizar que los cambios no rompan la funcionalidad existente de UDF.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
cpp
Área
backend
Tipo de issue
Refactorización
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
30/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.