Skip to content

fix(databricks): avoid creating UC volume in target schema#731

Closed
nehaaprasaad wants to merge 1 commit intogetnao:mainfrom
nehaaprasaad:fix/nao-crt-vol-dtb
Closed

fix(databricks): avoid creating UC volume in target schema#731
nehaaprasaad wants to merge 1 commit intogetnao:mainfrom
nehaaprasaad:fix/nao-crt-vol-dtb

Conversation

@nehaaprasaad
Copy link
Copy Markdown

@nehaaprasaad nehaaprasaad commented May 6, 2026

fix : #722

summary

  • Avoid creating Unity Catalog volumes in the user’s target schema during Databricks connection.

Changes

  • Do not pass schema_name into ibis.databricks.connect().
  • Use list_tables(database=...) in check_connection when schema_name is set.

Why

  • Ibis memtable volumes follow the session default schema; BI schemas are often read-only.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

This PR was auto-closed. Only contributors approved with lgtm can open PRs. Open an issue first.

Maintainers review auto-closed issues daily. Issues that do not meet the quality bar in CONTRIBUTING.md will not be reopened or receive a reply.

If a maintainer replies lgtmi, your future issues will stay open. If a maintainer replies lgtm, your future issues and PRs will stay open.

See CONTRIBUTING.md.

@github-actions github-actions Bot closed this May 6, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

🚀 Preview Deployment

URL https://pr-731-54908d7.preview.getnao.io
Commit 54908d7

⚠️ No LLM API keys configured - you'll see the API key setup flow when trying to chat.


Preview will be automatically removed when this PR is closed.

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.

[bug] nao creates a volume on Databricks under the target schema

1 participant