Skip to content

Properly handle verify parameter in TrinoHook #18770

Description

@eladkal

Body

This task refers to comment left in the Hook:

if extra.get('verify') is not None:
# Unfortunately verify parameter is available via public API.
# The PR is merged in the trino library, but has not been released.
# See: https://github.com/trinodb/trino-python-client/pull/31
trino_conn._http_session.verify = _boolify(extra['verify'])

The change of trinodb/trino-python-client#31 was released in trino version 0.301.0

The task:
Change the code + tests to work with the verify parameter directly. It will also require setting min version for trino in setup.py

Committer

  • I acknowledge that I am a maintainer/committer of the Apache Airflow project.

Metadata

Metadata

Assignees

Type

No type

Fields

No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions