Yuya Ebihara
Yuya Ebihara
Could you rebase on master to resolve conflicts?
Please fix CI failure and add a query based test to `BasePinotConnectorSmokeTest` ``` Error: TestDynamicTable.testQueryOptions:469 expected: "SET useMultistageEngine = true; SET skipUpsert = true; SELECT "FlightNum" FROM realtimeOnly_REALTIME LIMIT 50"...
@naman-patel Please handle https://github.com/trinodb/trino/pull/21902#discussion_r1596151758
I'm sorry, but I lean toward rejecting this PR. I faced unexpected query failure and a silent correctness issue while testing `enableNullHandling` locally. Also, this change allows unnecessary query options....
https://github.com/trinodb/trino/pull/22214 was merged. Can you try in your environment?
@sachleen0700 Please use [triple backticks](https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/creating-and-highlighting-code-blocks#fenced-code-blocks) and keep the indentation so that we can read easily.
Please feel free to reopen if the issue still happens in the latest version.
/test-with-secrets sha=bf0cf785188ed0cca97f6b18f4f750df035ebe75
@oskar-szwajkowski Can you open a new PR?
cc: @wendigo @hashhar