Unable to Edit

[samzuercher]

I am unable to make any edits in dbeaver.

1. Setup

• :memory: db with duckdb_jdbc-1.4.4.0.jar
• Run

INSTALL httpfs;LOAD httpfs;
INSTALL postgres;LOAD postgres;
INSTALL ducklake;LOAD ducklake;
INSTALL json;LOAD json;

CREATE OR REPLACE SECRET (
    TYPE gcs,
    KEY_ID 'GOOG',
    SECRET 'hh6w'
);

ATTACH 'ducklake:postgres:host=localhost dbname=bkt_gen_lake01 user=user1' AS bkt_gen (
    DATA_PATH 'gs://bucketname/some/duckdb/lake01'
);

-- note: in my sample data and table is already existing
CREATE TABLE IF NOT EXISTS "main"."Document" (
    "Path" VARCHAR,
    "Type" USMALLINT,
    "Date" TIMESTAMP,
    "Amount" DOUBLE,
    "Currency" VARCHAR,
    "Description" VARCHAR,
    "DescriptionAbbreviation" VARCHAR,
    "AccountCredit" USMALLINT,
    "AccountDebit" USMALLINT,
    "PartiesJson" JSON,
    "Ids" VARCHAR[],
    "Topic" USMALLINT[],
    "Generation" USMALLINT,
    "Created" TIMESTAMP,
    "Modified" TIMESTAMP,
    "Note" VARCHAR
);

2. Try to edit

1. Right-Click on a ducklake-Table -> View Data
2. Double Click in any Cell, e.g. Type, change a value and click 'Save' at the bottom of the UI
3. It shows:

Error synchronizing data with database

Can not bind statement parameter
  Unknown target type 2003

Am i doing something wrong or is Inline-Editing for ducklake not yet supported?

[staticlibs]

Hi, thanks for the report! The error is related to LIST parameters. I wonder if it works in general for you with tables containing only basic types, without arrays or lists?

[staticlibs]

This should be fixed in #588, after it is merged - can be checked using a Maven snapshot, also will be included into the upcoming 1.5.0.0.

With it I can add new rows with LIST columns in DBeaver. I cannot see the way to add multiple elements to the LIST though - not sure how is this supposed to work in DBeaver.

[samzuercher]

Thanks! I will try to test. I am not sure on array / list values in cells and only discussed quickly with gemini. It might be better to just use single values and put arrays in lookup tables. Not sure how this will impact query performance as querying e.g. on Topic would then need a join on a Topic-Table, which would be a second parquet file in ducklake. Will need to dig down a bit on this or get some answers.

[staticlibs]

@samzuercher

Just FYI, The snapshot with the fix was published, can be used with the following version:

<dependency>
    <groupId>org.duckdb</groupId>
    <artifactId>duckdb_jdbc</artifactId>
    <version>1.5-fd4d7ed0-SNAPSHOT</version>
  </dependency>

  <repository>
    <id>central-snapshots</id>
    <url>https://central.sonatype.com/repository/maven-snapshots/</url>
    <snapshots><enabled>true</enabled></snapshots>
  </repository>