4paradigm / 4paradigm/OpenMLDB

refactor JitManager and HybridSeJitWrapper for simplify

Open
#3,750 0 comments 0 reactions 1 assignee Claimed by @aceforeverd View on GitHub
enhancement execute-engine
Dominant language
C++
Stars
1.7k
Forks
331
Avg merge
12d 12h
Merged PRs (30d)
1

Description

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

Contributor guide

Open the contributing guide

Research direction

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.

Written by the indexing model from the issue text.

Assessment

Domain
backend, tooling
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.