Maxim Martynov
Maxim Martynov
It looks like `client.exists(path)` calls `client.propfind(path)` with default arguments: https://github.com/skshetry/webdav4/blob/7604957d93b875cbdeced34306fde0d961b9c239/src/webdav4/client.py#L549 And `client.propfind(path)` has argument `follow_redirects` with default value `False`: https://github.com/skshetry/webdav4/blob/7604957d93b875cbdeced34306fde0d961b9c239/src/webdav4/client.py#L307 Maybe `True` should be passed in `client.exists`?
Maybe this client should always follow redirects to handle both servers which require trailing slash and which require to trim it.
> That could be fixed, but we don't always know if the resource is a collection or not (eg: in `exists`). `exists` returns `True` for both file and directory, the...
Same as #2154, just moved to another branch and rebased
@Kludex could you take a look, please?
Hm, openlineage-python client minimal version is currently 1.16.0: https://github.com/apache/airflow/blob/ed57711866527d4e56ce195944b26607317b1f26/airflow/providers/openlineage/provider.yaml#L49 Why need adding these `try: ... except:` for handling versions `
> to test current latest code assuming that older versions of airlfow and older versions of providers might be installed How is this related to adding a bunch of `try..except`...
Ok, got it, thanks for clarifying!
Well, at least documentation should be updated first, to mention that this property is deprecated.
В unit тестах есть `test_jdbc_read_options_default`. Нужно сделать такой же для SQLOptions.