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.

Sign up for GitHub

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

Jump to bottom

[Subtask] Adapt the Hadoop catalog to the CatalogOperationDispatcher #1667

Closed
Tracked by #1241
jerryshao opened this issue Jan 24, 2024 · 0 comments · Fixed by #1714
Closed
Tracked by #1241

[Subtask] Adapt the Hadoop catalog to the CatalogOperationDispatcher #1667

jerryshao opened this issue Jan 24, 2024 · 0 comments · Fixed by #1714
Assignees
@jerryshao
Milestone
Gravitino 0.4.0

Comments

@jerryshao
Copy link
Contributor

Describe the subtask

This issue tracks the work of adapting the implemented Hadoop catalog to the CatalogOperationDispatcher.

Parent issue

#1241
@jerryshao jerryshao added this to the Gravitino 0.4.0 milestone Jan 24, 2024
@jerryshao jerryshao self-assigned this Jan 24, 2024
This was referenced Jan 24, 2024
Closed
Merged
@coolderli coolderli mentioned this issue Jan 26, 2024
Closed
jerryshao added a commit that referenced this issue Jan 29, 2024
@jerryshao
[#1667] feat(hadoop-catalog, core): Adapt the fileset catalog to Cata…
912f8d7 
…logOperationsDispatcher (#1714)

### What changes were proposed in this pull request?

This PR add the Hadoop catalog adaption to the
CatalogOperationsDispatcher

### Why are the changes needed?

Part of fileset catalog support work

Fix: #1667

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

No.

### How was this patch tested?

UTs.
@jerryshao jerryshao mentioned this issue Jan 29, 2024
Merged
jerryshao added a commit that referenced this issue Jan 29, 2024
@jerryshao
[MINOR] fix(hadoop-catalog): Fix building issue after #1667 is merged. (
48ef601 
#1753)

### What changes were proposed in this pull request?

This PR is a followup PR to fix the building issue after #1667 is
merged.

### Why are the changes needed?

Fix building issue.

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

No.

### How was this patch tested?

Existing UTs.
@jerryshao jerryshao mentioned this issue Jan 29, 2024
Merged
jerryshao added a commit that referenced this issue Jan 29, 2024
@jerryshao
[#1760] Fix(core): Fix bug when introducing fileset api in CatalogOpe…
348b47f 
…rationDispatcher (#1762)

### What changes were proposed in this pull request?

Using the correct PropertiesMetadata to check the value when doing
operations.

### Why are the changes needed?

This is bug introduced in #1667 , fix it here.

Fix: #1706 

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

No.

### How was this patch tested?

Add UTs.
ch3yne pushed a commit to ch3yne/gravitino that referenced this issue Jan 29, 2024
@jerryshao @ch3yne
[apache#1667] feat(hadoop-catalog, core): Adapt the fileset catalog t…
547e9b3 
…o CatalogOperationsDispatcher (apache#1714)

### What changes were proposed in this pull request?

This PR add the Hadoop catalog adaption to the
CatalogOperationsDispatcher

### Why are the changes needed?

Part of fileset catalog support work

Fix: apache#1667

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

No.

### How was this patch tested?

UTs.
ch3yne pushed a commit to ch3yne/gravitino that referenced this issue Jan 29, 2024
@jerryshao @ch3yne
[MINOR] fix(hadoop-catalog): Fix building issue after apache#1667 is …
0710ef8 
…merged. (apache#1753)

### What changes were proposed in this pull request?

This PR is a followup PR to fix the building issue after apache#1667 is
merged.

### Why are the changes needed?

Fix building issue.

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

No.

### How was this patch tested?

Existing UTs.
ch3yne pushed a commit to ch3yne/gravitino that referenced this issue Jan 29, 2024
@jerryshao @ch3yne
[apache#1760] Fix(core): Fix bug when introducing fileset api in Cata…
23b045e 
…logOperationDispatcher (apache#1762)

### What changes were proposed in this pull request?

Using the correct PropertiesMetadata to check the value when doing
operations.

### Why are the changes needed?

This is bug introduced in apache#1667 , fix it here.

Fix: apache#1706 

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

No.

### How was this patch tested?

Add UTs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jerryshao jerryshao

Labels
None yet
Projects
None yet
Milestone
Gravitino 0.4.0
1 participant
@jerryshao