Skip to content

chore: Add qualifiedName support for Builtin*#630

Merged
stephenamar-db merged 1 commit intodatabricks:masterfrom
He-Pin:prefix
Mar 4, 2026
Merged

chore: Add qualifiedName support for Builtin*#630
stephenamar-db merged 1 commit intodatabricks:masterfrom
He-Pin:prefix

Conversation

@He-Pin
Copy link
Contributor

@He-Pin He-Pin commented Mar 4, 2026

refs: #629

@jam01 Wdyt

@jam01
Copy link
Contributor

jam01 commented Mar 4, 2026

This is good. Wonder how we can tie it from the FunctionBuilder functions...

@He-Pin He-Pin marked this pull request as draft March 4, 2026 16:47
@He-Pin He-Pin marked this pull request as ready for review March 4, 2026 16:55
@He-Pin
Copy link
Contributor Author

He-Pin commented Mar 4, 2026

I think now, you can attach the xtr as prefix, wdyt @jam01 ,thanks for the input,nice idea.

@jam01
Copy link
Contributor

jam01 commented Mar 4, 2026

Looks good to me!

@He-Pin
Copy link
Contributor Author

He-Pin commented Mar 4, 2026

@jam01 Seems like you are building xtr on sjsonnet 0.6.0? I think we should add more document to this project.

@jam01
Copy link
Contributor

jam01 commented Mar 4, 2026

Yes, I'm adding a couple of things on top of 0.6.0.

What do you mean by document? To my fork?

@stephenamar-db stephenamar-db merged commit a95b6e2 into databricks:master Mar 4, 2026
12 checks passed
@He-Pin He-Pin deleted the prefix branch March 4, 2026 18:33
@He-Pin
Copy link
Contributor Author

He-Pin commented Mar 4, 2026

@jam01 I mean the extension point of sjsonnet, our data mapping pipeline is built on sjsonnet too.

@jam01
Copy link
Contributor

jam01 commented Mar 4, 2026

Which data mapping is that?

@He-Pin
Copy link
Contributor Author

He-Pin commented Mar 4, 2026

We are using jsonnet to transform our mcp server data model with different data source, and export that as a tool

@jam01
Copy link
Contributor

jam01 commented Mar 4, 2026

The other changes I've made would require a larger surface and I haven't quite figured out how to hide some things behind feature flags, but I'm open to talk about them.

Would be interested to read more about your project as well.

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