Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Java] LargeListView Implementation for C Data Interface #41585

Closed
Tracked by #41271 ...
vibhatha opened this issue May 8, 2024 · 1 comment
Closed
Tracked by #41271 ...

[Java] LargeListView Implementation for C Data Interface #41585

vibhatha opened this issue May 8, 2024 · 1 comment

Comments

@vibhatha
Copy link
Collaborator

vibhatha commented May 8, 2024

Describe the enhancement requested

Java ListView integration will contain multiple steps as described in #41271.
Thus supporting C Data Interface is an important component for LargeListViewVector.

Component(s)

Java

@vibhatha vibhatha self-assigned this May 9, 2024
lidavidm pushed a commit that referenced this issue May 16, 2024
### Rationale for this change

Apache Arrow format defines ListView and this has been introduced into other language bindings and the objective of this PR is to provide the initial ListView support to Java by adding `ListViewVector`. 

### Non-Goals 

The following list of issues propose the extended work depending on this PR. They were separated to streamline the implementation process. 

- [ ] #41272
- [ ] #41286
- [ ] #41290
- [ ] #41288
- [ ] #41289
- [ ] #41269
- [ ] #41291
- [ ] #41292
- [ ] #41270
- [ ] #41293
- [ ] #41294
- [ ] #41569
- [ ] #41570
- [ ] #41584
- [ ] #41585

### Are these changes tested?

Yes

### Are there any user-facing changes?

No

* GitHub Issue: #41287

Lead-authored-by: Vibhatha Abeykoon <vibhatha@gmail.com>
Co-authored-by: Vibhatha Lakmal Abeykoon <vibhatha@gmail.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
vibhatha added a commit to vibhatha/arrow that referenced this issue May 25, 2024
### Rationale for this change

Apache Arrow format defines ListView and this has been introduced into other language bindings and the objective of this PR is to provide the initial ListView support to Java by adding `ListViewVector`. 

### Non-Goals 

The following list of issues propose the extended work depending on this PR. They were separated to streamline the implementation process. 

- [ ] apache#41272
- [ ] apache#41286
- [ ] apache#41290
- [ ] apache#41288
- [ ] apache#41289
- [ ] apache#41269
- [ ] apache#41291
- [ ] apache#41292
- [ ] apache#41270
- [ ] apache#41293
- [ ] apache#41294
- [ ] apache#41569
- [ ] apache#41570
- [ ] apache#41584
- [ ] apache#41585

### Are these changes tested?

Yes

### Are there any user-facing changes?

No

* GitHub Issue: apache#41287

Lead-authored-by: Vibhatha Abeykoon <vibhatha@gmail.com>
Co-authored-by: Vibhatha Lakmal Abeykoon <vibhatha@gmail.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
lidavidm pushed a commit that referenced this issue Aug 24, 2024
### Rationale for this change

C Data Interface is missing `ListView` and `LargeListView` after recently merging core functionalities.

Also closes;

- [x] #41585

### What changes are included in this PR?

This PR includes C Data interface related component additions to `ListView` and `LargeListView` along with the corresponding test cases. 

### Are these changes tested?

Yes

### Are there any user-facing changes?

No
* GitHub Issue: #41584

Authored-by: Vibhatha Abeykoon <vibhatha@gmail.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
@vibhatha
Copy link
Collaborator Author

Issue resolved by pull request 43686
#43686

@lidavidm lidavidm added this to the 18.0.0 milestone Aug 28, 2024
mapleFU pushed a commit to mapleFU/arrow that referenced this issue Sep 3, 2024
…pache#43686)

### Rationale for this change

C Data Interface is missing `ListView` and `LargeListView` after recently merging core functionalities.

Also closes;

- [x] apache#41585

### What changes are included in this PR?

This PR includes C Data interface related component additions to `ListView` and `LargeListView` along with the corresponding test cases. 

### Are these changes tested?

Yes

### Are there any user-facing changes?

No
* GitHub Issue: apache#41584

Authored-by: Vibhatha Abeykoon <vibhatha@gmail.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants