Zeashan

Results 10 comments of Zeashan

As discussed with @zsxwing there is no SQL support for generated columns, therefore I did not add support for it here. It is a minor change.

> I missed this case, before I re-request review I will add it.

Some related thoughts after reading this... The underlying error that matches this text lives in a variable named `DELTA_MULTIPLE_SOURCE_ROW_MATCHING_TARGET_ROW_IN_MERGE` Judging by that and by where it's called, it's a good...

Here's some additional thoughts: * We will always enforce all delta. properties to be case sensitive. * We will add a new feature flag `allowCaseInsensitiveProperties.enabled`, which would be to allow...

> > Hi @dennyglee, what about support for displaying DDL of delta tables (`SHOW CREATE TABLE`) #1032 #1255 > > Good call out @keen85 - let me check with @zpappa...

@zsxwing the ability to do this via file paths is now supported by this PR

Let's not implement this right now. There are many higher concerns.

So I see a small problem here. If we allow a customer to modify the default database/catalog when sync'ing tables (which we should), this will not be a useful step...

The problem with headless things is that they lack intelligence clearly. If the headless automation leaves things in a broken state, it provides no value. The only way this is...