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

[#1047] improvement(core): Fix the issue of StringIdentifier in metalake property #1188

Merged
merged 1 commit into from
Dec 18, 2023
Merged

[#1047] improvement(core): Fix the issue of StringIdentifier in metalake property #1188

merged 1 commit into from
Dec 18, 2023

Commits on Dec 18, 2023

  1. [#1047] improvement(core): Fix the issue of StringIdentifier in metal… 

    …ake property (#1163)

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

Remove the `stringId` from metalake property map. 

### Why are the changes needed?

`gravitino.identifier` is generated internally and should not be visible
to users.

Fix: #1047 

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

N/A

### How was this patch tested?

Existing UTs can cover it.
    @yuqi1129 @web-flow
    yuqi1129 authored and web-flow committed Dec 18, 2023
    Configuration menu
    Copy the full SHA
    4decff0 View commit details
    Browse the repository at this point in the history