4paradigm / 4paradigm/OpenMLDB

cast to int64_t in GetAsInt32()

Abierto
#3,675 1 comentario 0 reacciones 0 asignados Ver en GitHub
enhancement minor
Lenguaje dominante
C++
Estrellas
1.7k
Forks
331
Merge medio
12 d 12 h
PR fusionados (30 d)
1

Descripción

https://github.com/4paradigm/OpenMLDB/blob/0c2592da4da94d34804560ada03dd743aa7f25b5/hybridse/include/node/sql_node.h#L975-L991

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Look at hybridse/include/node/sql_node.h lines 975-991, specifically the GetAsInt32() method. The issue suggests adding a cast to int64_t. Check the surrounding code to understand the data types involved and ensure the cast is safe. Run any existing tests related to SQL node parsing or integer conversions to verify the change.

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

Evaluación

Stack tecnológico
cpp
Área
databases
Tipo de issue
Refactorización
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.