4paradigm / 4paradigm/OpenMLDB

Remove the memcpy for UnsafeRowOpt

Abierto
#1,893 0 comentarios 0 reacciones 1 asignado Reclamado por @tobegit3hub Ver en GitHub
enhancement
Lenguaje dominante
C++
Estrellas
1.7k
Forks
331
Merge medio
12 d 12 h
PR fusionados (30 d)
1

Descripción

Now we need to memcpy the entire row for UnsafeRowOpt because of usage of row header. It would be great to remove the memcpy for UnsafeRowOpt by not depending on row header in some cases.

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Look at the UnsafeRowOpt implementation, likely in a file like core/row_opt.cpp or similar. Understand the current row header usage and memcpy. Identify cases where the header dependency can be avoided. Test changes with existing unit tests for row operations and performance benchmarks.

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

Evaluación

Stack tecnológico
cpp
Área
databases, performance
Tipo de issue
Refactorización
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.