Jarek Potiuk comments

Results 1573 comments of


                                            Jarek Potiuk

Masking configuration values irrelevant to DAG author

I think we generally should provide rather comprehensive guide (at least with some links) to our contributors how to setup the venv now (Breeze is covered because it uses workspace...

Masking configuration values irrelevant to DAG author

Looks like all the comments are resolved and address by @amoghrajesh . LGTM. @kaxil @ashb - are you ok with merging it?

Masking configuration values irrelevant to DAG author

> @amoghrajesh , shouldn't the list of sensitive_config_values be updated with the list of keys that need to redact ? Line: https://github.com/amoghrajesh/airflow/blob/840ea3efb9533837e9f36b75fa527a0fbafeb23a/airflow/configuration.py#L121 Not really @saurabhb-dev , It's even better to...

Snowflake SQL API hook Retry Logic

> Hi @astro-anand do you happen to know when this will be released? We usually release provider's every 2 weeks (See the README). We have one release in progress (see...

Adding `prevent_self_review` property to `Repository.createEnvironment`

would be cool to merge it in :)

Adding `prevent_self_review` property to `Repository.createEnvironment`

Yes. It would be really useful to get it in the release

Deprecate legacy `setup.py develop` mechanism for `pip install --editable`

Comment from a side. I am really sorry for it to be a long message, I just wanted to provide a bit more context. So for those who would like...

Deprecate legacy `setup.py develop` mechanism for `pip install --editable`

> @potiuk What you are saying makes sense for wider-reaching changes like the historic switch from the legacy resolver, however, I don't think it's as applicable here. Since: What you...

Make conn id parameters templated in GenericTransfer and also allow passing hook parameters like in BaseSQLOperator

Looks like there is some problem with minimum version of one of microsoft dependencies. Bringing it down, brings opentelemetry dependency down and this seems to break imports. Unfold the failng...

Make conn id parameters templated in GenericTransfer and also allow passing hook parameters like in BaseSQLOperator

There are some compatibility Airflow 3 / Airflow 2 issues that need to be addressed :)