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

[BUG]: Notebook linter includes lower cells within linting context #3543

Open
1 task done
JCZuurmond opened this issue Jan 17, 2025 · 0 comments · May be fixed by #3535
Open
1 task done

[BUG]: Notebook linter includes lower cells within linting context #3543

JCZuurmond opened this issue Jan 17, 2025 · 0 comments · May be fixed by #3535
Assignees
Labels
bug Something isn't working needs-triage

Comments

@JCZuurmond
Copy link
Member

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

See the following notebook

# Databricks notebook source

table_name = "old.things"  # Migrated table according to the migration index

# COMMAND ----------

spark.table(table_name)

# COMMAND ----------

table_name = "not_migrated.table"  # NOT a migrated table according to the migration index

The spark.table(table_name) is reading a migrated table hence we expect a deprecation warning to be given that says to use the migrated table. However, the linter does not yield such a warning because it uses the table_name from the lowest cell.

Expected Behavior

A cell should only include the context from itself cell, the cells above and possible context from %run references

Steps To Reproduce

No response

Cloud

Azure

Operating System

macOS

Version

latest via Databricks CLI

Relevant log output

@JCZuurmond JCZuurmond added bug Something isn't working needs-triage labels Jan 17, 2025
@JCZuurmond JCZuurmond self-assigned this Jan 17, 2025
@JCZuurmond JCZuurmond added this to UCX Jan 17, 2025
@github-project-automation github-project-automation bot moved this to Todo in UCX Jan 17, 2025
@JCZuurmond JCZuurmond linked a pull request Jan 17, 2025 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs-triage
Projects
Status: Todo
Development

Successfully merging a pull request may close this issue.

1 participant