4paradigm / 4paradigm/OpenMLDB

refactor JitManager and HybridSeJitWrapper for simplify

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

Descripción

## Simplify
**Create** > **Init** , hides things like **InitSymbols**

## Error control

- returns `base::Status` on error, not `bool`

## Lifetime control: no manual delete in java

The `HybridSeJitWrapper` in hybridse-sdk is a proxy class to C pointer `HybridSeJitWrapper`. This requires extra config in swig so the proxy class will automatically release C resource from GC

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Look at the JitManager and HybridSeJitWrapper classes in the codebase, likely in a jit/ or sdk/ directory. Understand how InitSymbols and manual delete are currently handled. The goal is to refactor creation to hide InitSymbols, change error returns to base::Status, and configure SWIG for automatic lifetime management of the C pointer in the Java proxy. Check existing SWIG interface files for HybridSeJitWrapper.

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

Evaluación

Área
backend, tooling
Tipo de issue
Refactorización
Dificultad
4/5
Tiempo estimado
3-5 días
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.