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

[Subtask] Add ColumnImpl in API module #1630

Closed
Tracked by #1292
mchades opened this issue Jan 22, 2024 · 0 comments · Fixed by #1631
Closed
Tracked by #1292

[Subtask] Add ColumnImpl in API module #1630

mchades opened this issue Jan 22, 2024 · 0 comments · Fixed by #1631
Assignees

Comments

@mchades
Copy link
Contributor

mchades commented Jan 22, 2024

Describe the subtask

Add ColumnImpl in API module

Parent issue

#1292

@mchades mchades self-assigned this Jan 22, 2024
jerryshao pushed a commit that referenced this issue Jan 23, 2024
### What changes were proposed in this pull request?

Add `columnImpl` in API module

### Why are the changes needed?

For client usage

Fix: #1630 

### Does this PR introduce _any_ user-facing change?

yes, the client now can use the helper method in `Column` API to get a
column instance

### How was this patch tested?

UTs added
mchades added a commit to mchades/gravitino that referenced this issue Jan 24, 2024
### What changes were proposed in this pull request?

Add `columnImpl` in API module

### Why are the changes needed?

For client usage

Fix: apache#1630 

### Does this PR introduce _any_ user-facing change?

yes, the client now can use the helper method in `Column` API to get a
column instance

### How was this patch tested?

UTs added
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant