Harshal Sheth

@mayurinehate looks like there's a conflict on this one

@bda618 I'm glad the changes from the other PR fixed the status stuff LGTM = looks good to me :)

@bda618 usually `pytest tests/ --update-golden-files` will do the trick

CI failures appear unrelated and caused by https://github.com/datahub-project/datahub/pull/7059. We should be ok to merge this as long as the 3.10 tests pass.

@frsann two high-level things before I get into the review 1. My understanding from reading the snowflake docs is that tags in snowflake are global within an account. If that's...

@looppi would it be accurate to say that https://github.com/datahub-project/datahub/pull/7519 is a superset of the changes here? I'm planning on merging that one soon cc @aezomz, but wanted to make sure...

@looppi thanks for the confirmation I'm closing this PR in favor of https://github.com/datahub-project/datahub/pull/7519

~~Currently blocked on https://github.com/googleapis/python-bigquery-sqlalchemy/issues/500.~~ Now blocked on https://github.com/googleapis/python-bigquery-sqlalchemy/pull/543

Looks like we're somehow pinning pyarrow to an old version, which doesn't have a pre-built binary for python 3.11. Ideally we should loosen our deps, but we can also add...

Blocked because of https://github.com/feast-dev/feast/issues/3510