[Java] Make JniWrapper native method be public
- Vorherrschende Sprache
- Java
- Sterne
- 94
- Forks
- 152
- Ø Merge
- 3 T. 16 Std.
- Gemergte PRs (30 T.)
- 11
Beschreibung
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.*
Beitragsleitfaden
Rechercherichtung
Beginne damit, JniWrapper zu finden und seine Methoden auf Paketebene sowie die aktuellen Aufrufer zu überprüfen. Kläre, ob der Umfang nur darin besteht, diese Methoden public zu machen, oder auch darin, die Klasse in UnsafeJniWrapper umzubenennen; als erledigt gilt die Aufgabe, wenn die vereinbarte API für externe Pakete zugänglich ist und weiterhin für die Integration mit Gandiva und Drill geeignet bleibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- java
- Bereich
- api
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 25/100