Results 208 comments of Pieter Noordhuis

@ericfeunekes Can you update the PR summary to match the current state of the PR?

@ericfeunekes Thanks for signing the CLA. This PR can proceed. Can you update the PR summary to match the current state of the PR?

Thanks for addressing the issue and submitting the bump, @vedantmgoyal9. We'll hold on for a while to merge this because we need CLAs on this repo and am checking if...

@vedantmgoyal9 Thanks again for fixing upstream and the bump here. We addressed the underlying issue as well as pinning the Komac version by calling Komac directly (#2228).

Thanks for posting the issue. You're right, this is not implemented. The reasoning for not including it out of the gate is that seeing this metadata is primarily helpful for...

@ksummerlin It is possible to look up a warehouse ID if one exists today, see [the docs](https://docs.databricks.com/aws/en/dev-tools/bundles/variables#retrieve-an-objects-id-value) or [this example](https://github.com/databricks/bundle-examples/blob/bdaed691c4ac66e66159d91db572bb838245cad4/knowledge_base/dashboard_nyc_taxi/databricks.yml#L7-L12).

Thanks for reporting the issue. Does it actually save a file, and if so, what's the filename you see on disk? We should look into the filename normalization logic.

I reproduced the issue and observed that on Windows it writes to a file named "src/Test DLT", ignoring everything from ":". The change in #3090 performs filename normalization and converts...