Piotr Findeisen
Piotr Findeisen
> I see that there are connectors in Trino which allow specifying "primary_key" table properties, so its not a new idea. In case of RDBMSes, we rely on separate engine...
This PR does too much IMO, changing places where improvement can be expected, but also places where it rather cannot. Example here: https://github.com/trinodb/trino/pull/27343#discussion_r2599221741 but it looks like a pattern rather...
This PR lacks description that motivates the change. It also changes concurrent code in ways that could affect what the code does. It is normal for a change to break...
> The Teradata connector module has both unit tests and integration tests. > The integration tests require access to a [Teradata ClearScape Analytics™ Experience](https://clearscape.teradata.com/sign-in). How will these tests be run...
is the build failure related?
> 2\. getting the exception in the second attempt is unfortunate but it is expected if you view this from "is it working as implemented?" perspective, then yes, it does...
Do i understand correctly that in OP context, for a `platform_id = 1 OR platform_id = 2 OR platform_id = 3 OR platform_id = 4` filter we would produce a...
> We heard back from OpenText now and are working towards a solution. Great, what's the timeline? > Worst case for now we ship the connector without test coverage. That...
> Most often we experienced this test failing due to timeout waiting on the barrier, > the reason behind this for pooled connectors is connection timeout, Or are we witnessing...
Maybe this test is more problems than it's worth....