search

treeverse / airflow-provider-lakeFS

lakeFS airflow operator
Apache License 2.0
26 stars 5 forks source link

Add client ID to lakeFS client connections #76

Closed arielshaqed closed 1 year ago

arielshaqed commented 1 year ago

Format: "lakefs-airflow-provider (VERSION)".

Fixes #68.

arielshaqed commented 1 year ago

You can see the client string passed in the provider test - look for "lakeFS logs", it appears as "lakefs-airflow-provider (0.46.3)".

I don't think we should spend time on preventing dev version from misreporting themselves as the latest release.

arielshaqed commented 1 year ago

@lynnro314 for Airflow, @nopcoder to check I didn't do something Pythonically stupid. I will pull with a single approval unless you ask me not to pull.

arielshaqed commented 1 year ago

Thanks! I switched to using your method of retrieving the package version. PTAL...

arielshaqed commented 1 year ago

Thanks!