4paradigm / 4paradigm/OpenMLDB

refactor JitManager and HybridSeJitWrapper for simplify

未关闭
#3,750 0 条评论 0 个 reaction 已指派 1 人 已被 @aceforeverd 认领 在 GitHub 查看
enhancement execute-engine
主要语言
C++
星标
1.7k
派生
331
平均合并
12 天 12 小时
30 天内合并 PR
1

描述

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

贡献指南

打开贡献指南

调研方向

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.

由索引模型根据 Issue 内容生成。

评估

领域
backend, tooling
Issue 类型
重构
难度
4/5
预计耗时
3-5 天
活跃度
停滞
描述清晰度
基本清楚
新手友好度
45/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。