Results 30 comments of David Blain

FYI: I've seen accross different PR that following test is sometimes randomly failing: ``` =================================== FAILURES =================================== _______ TestCeleryExecutor.test_celery_integration[redis://redis:6379/0] _______ self = broker_url = 'redis://redis:6379/0' @pytest.mark.flaky(reruns=3) @pytest.mark.parametrize("broker_url", _prepare_test_bodies()) def test_celery_integration(self,...

> A PowerBi / Microsoft Fabric provider would be really nice ! We (Infrabel) started to work on that, via @dabla's MsGraph Operators. > > @ambika-garg I contacted you on...

> As far as i remember, we recently discuss this case in slack. I guess better introduce `sa_uri` property (or some similar naming) which returns [`sqlalchemy.engine.URL`](https://docs.sqlalchemy.org/en/14/core/engines.html#sqlalchemy.engine.URL) into the `DbApiHook` without...

> There is new property which intends to returns SQL Alchemy URL, current behaviour of `get_url` is ambiguous and non-all hook return SA URI here, for some of the DB...

This is how I done it in my custom SQLRowsInsertOperator: ``` @cached_property def connection(self) -> Connection: return BaseHook.get_connection(self.conn_id) # type: ignore @cached_property def scheme(self) -> str: return self.connection.extra_dejson["sqlalchemy_scheme"] @cached_property def...

Here is a screenshot of the updated HTTP (and Livy) connection form. I had to update the javascript file of connection_form.js to not only enable CodeMirror on the extra field...

I think this one is finally ready

> > I think this one is finally ready > > can you undraft it then ? Unfortunately I can’t as it’s not my pull request, @Joffreybvn has to do...

> > I think this one is finally ready > > can you undraft it then ? Unfortunately I can’t as it’s not my pull request, @Joffreybvn has to do...

It’s a bit hard to split as all modifications are linked to each other. I was already happy I was able to pull it off imho ☺️