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

[#4812] fix(core): Make ensure the schema exists before creating tables and topics #4826

Merged
merged 1 commit into from
Aug 30, 2024

Commits on Aug 30, 2024

  1. [apache#4812] fix(core): Make ensure the schema exists before creatin…

    …g tables and topics (apache#4818)
    
    ### What changes were proposed in this pull request?
    If the schema is not created by Gravitino, the Gravitino will lack the
    metadata in the backend storage.
    If we create a table in this schema, the storage won't contain the
    metadata. So it will fail to set owner. Because the storage won't store
    the table. Because the storage won't contain schema id.
    
    This won't bring too much performance cost. Because loadSchema will use
    read lock after first loading. If we have cache, we could be more quick.
    
    ### Why are the changes needed?
    
    Fix: apache#4812
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    Added a ut.
    jerqi committed Aug 30, 2024
    Configuration menu
    Copy the full SHA
    d29190d View commit details
    Browse the repository at this point in the history