databrickslabs / lsql

Lightweight SQL execution wrapper only on top of Databricks SDK
https://pypi.org/project/databricks-labs-lsql/
Other
7 stars 1 forks source link

Fix tablev1 spec #196

Closed JCZuurmond closed 2 weeks ago

JCZuurmond commented 2 weeks ago

Thought that #190 introduces the TableV1Spec, the integration test show the tables, however upon further inspection, the table is automatically converted to a new table

Screenshot 2024-06-26 at 09 41 04

Not a problem on itself, though we would like to avoid the warning. However, it became a problem when introducing columns that https link to resources. Then we get an error that the widget definition is invalid.

For testing, I trimmed it down to:

Screenshot 2024-06-26 at 09 45 14
JCZuurmond commented 2 weeks ago

Solved by always adding the invisibleColumns to the body

Screenshot 2024-06-27 at 09 30 32