apache / apache/arrow-java

[Java] Make JniWrapper native method be public

Aberta
#360 3 comentários 0 reações 0 responsáveis Ver no GitHub
Type: enhancement
Linguagem predominante
Java
Estrelas
94
Forks
152
Merge médio
3d 16h
PRs com merge (30d)
11

Descrição

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.*

Guia de contribuição

Abrir o guia de contribuição

Direção de pesquisa

Comece localizando JniWrapper e revisando seus métodos no nível do pacote e os chamadores atuais. Esclareça se o escopo consiste apenas em tornar esses métodos públicos ou também em renomear a classe para UnsafeJniWrapper; considera-se concluído quando a API acordada estiver acessível a pacotes externos e continuar adequada à integração com Gandiva e Drill.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
java
Domínio
api
Tipo de issue
Funcionalidade
Dificuldade
4/5
Tempo estimado
3-5 dias
Status de atividade
Estagnada
Clareza
Precisa de esclarecimento
Facilidade para iniciantes
25/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.