4paradigm / 4paradigm/OpenMLDB

refactor JitManager and HybridSeJitWrapper for simplify

Aperta
#3,750 0 commenti 0 reazioni 1 assegnatario Rivendicata da @aceforeverd Vedi su GitHub
enhancement execute-engine
Lingua principale
C++
Stelle
1.7k
Fork
331
Merge medio
12g 12h
PR unite (30g)
1

Descrizione

## 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

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

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.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Ambito
backend, tooling
Tipo di issue
Refactoring
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.