apache / apache/arrow-java

[Java] Make JniWrapper native method be public

未关闭
#360 3 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
Type: enhancement
主要语言
Java
星标
94
派生
152
平均合并
3 天 16 小时
30 天内合并 PR
11

描述

The goal is to integrate Gandiva into apache Drill project. Now drill and arrow has some differences at the column in memory representation. Drill has a 2.0 plan to integrate arrow. Now I want to do some prior work to integrate arrow lib into drill project. The first thing is to make JniWrapper's package level methods be public ones. Maybe we can rename this classes as UnsafeJniWrapper like java's Unsafe class to allow others to invoke these methods directly.

So what's your opinion about this advice ? If ok , I will submit a corresponding PR.

**Reporter**: [Weijie Tong](https://issues.apache.org/jira/browse/ARROW-4819)

**Note**: *This issue was originally created as [ARROW-4819](https://issues.apache.org/jira/browse/ARROW-4819). Please see the [migration documentation](https://github.com/apache/arrow/issues/14542) for further details.*

贡献指南

打开贡献指南

调研方向

首先定位 JniWrapper,并检查其包级方法及当前调用方。明确范围是否仅包括将这些方法设为 public,还是还包括将类重命名为 UnsafeJniWrapper;当约定的 API 可供外部包访问,并且仍适用于与 Gandiva 和 Drill 的集成时,即视为完成。

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

评估

技术栈
java
领域
api
Issue 类型
功能
难度
4/5
预计耗时
3-5 天
活跃度
停滞
描述清晰度
需要澄清
新手友好度
25/100

把新 issue 发到你的邮箱

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