Skip to content

Conversation

wendigo
Copy link
Contributor

@wendigo wendigo commented Jan 30, 2025

Description

Additional context and related issues

Release notes

( ) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text:

## Section
* Fix some things. ({issue}`issuenumber`)

@cla-bot cla-bot bot added the cla-signed label Jan 30, 2025
@wendigo wendigo requested review from ebyhr and losipiuk January 30, 2025 06:32
@github-actions github-actions bot added the iceberg Iceberg connector label Jan 30, 2025
@wendigo wendigo force-pushed the serafin/updates-jan-30 branch from 391935d to 350f5ce Compare January 30, 2025 08:14
@wendigo
Copy link
Contributor Author

wendigo commented Jan 30, 2025

@ebyhr I've dropped the escape == null condition from DuckDbClient. It seems that new driver is actually setting escape correctly.

@ebyhr
Copy link
Member

ebyhr commented Jan 30, 2025

@wendigo Thanks, duckdb/duckdb-java#89 fixed the issue.

@wendigo wendigo merged commit f5790de into master Jan 30, 2025
110 of 111 checks passed
@wendigo wendigo deleted the serafin/updates-jan-30 branch January 30, 2025 09:22
@github-actions github-actions bot added this to the 470 milestone Jan 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed iceberg Iceberg connector

Development

Successfully merging this pull request may close these issues.

2 participants