Skip to content

[MINOR] Improve(lance-namespace): Slightly improve the configuration and build script#8918

Merged
mchades merged 4 commits intoapache:branch-lance-namepspace-devfrom
jerryshao:lance-minor-fix
Oct 28, 2025
Merged

[MINOR] Improve(lance-namespace): Slightly improve the configuration and build script#8918
mchades merged 4 commits intoapache:branch-lance-namepspace-devfrom
jerryshao:lance-minor-fix

Conversation

@jerryshao
Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

This PR improves the configuration key and also the build kts for lance namespace service.

Why are the changes needed?

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Local test.

@jerryshao jerryshao closed this Oct 27, 2025
@jerryshao jerryshao reopened this Oct 27, 2025
@jerryshao jerryshao changed the base branch from main to dev-lance-namespace October 27, 2025 07:05
@jerryshao jerryshao changed the base branch from dev-lance-namespace to branch-lance-namepspace-dev October 27, 2025 07:06
@jerryshao jerryshao requested review from mchades and yuqi1129 October 27, 2025 07:06
@jerryshao jerryshao self-assigned this Oct 27, 2025
# The uri of the Lance REST service gravitino namespace backend
gravitino.lance-rest.gravitino.uri = http://localhost:8090
# The metalake name used for Lance REST service gravitino namespace backend
#gravitino.lance-rest.gravitino.metalake-name =
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the default value for metalake null?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's no default value, user should configure this if they want to use Lance namespace service.

Copy link
Copy Markdown
Contributor

@mchades mchades left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@mchades mchades merged commit bedc522 into apache:branch-lance-namepspace-dev Oct 28, 2025
28 of 30 checks passed
jerryshao added a commit to jerryshao/gravitino that referenced this pull request Nov 11, 2025
…and build script (apache#8918)

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

This PR improves the configuration key and also the build kts for lance
namespace service.

### Why are the changes needed?

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

No.

### How was this patch tested?

Local test.
youngyjd pushed a commit to youngyjd/gravitino that referenced this pull request Nov 19, 2025
…and build script (apache#8918)

This PR improves the configuration key and also the build kts for lance
namespace service.

No.

Local test.
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 this pull request may close these issues.

3 participants