4paradigm / 4paradigm/OpenMLDB

CREATE FUNCTION: dlopen/dlclose not need to mutex lock handler

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

描述

`dlopen` returns handler that has reference count internally, we do not need to lock with the `map handler>`, the same for `dlclose`. A simple `map handler>` is cleaner.

贡献指南

打开贡献指南

调研方向

Look for the CREATE FUNCTION implementation and the map handling dlopen/dlclose handlers. The issue suggests removing mutex locks around the filename-to-handler map and using a simpler fnname-to-handler map. Start by finding the relevant code in the source tree, likely around function loading or dynamic library management. Understand the current locking mechanism and reference counting to ensure thread safety without the mutex. Verify changes by building and testing the function loading functionality.

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

评估

技术栈
cpp
领域
databases
Issue 类型
重构
难度
3/5
预计耗时
1-2 天
活跃度
停滞
描述清晰度
基本清楚
新手友好度
45/100

把新 issue 发到你的邮箱

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