0.10.0 Regression - Cannot parse metadata

[harel-e]

Starting with DuckDB 0.10.0, I can no longer use the extension to scan iceberg files

select * from iceberg_scan('s3://zones/iceberg/test/zones_43_1b7d1e67-8f53-42ab-ba55-a66f438f344f/metadata/00006-0d6ca258-8039-446f-843a-69e21344f272.metadata.json');

Error: IO Error: Invalid field found while parsing field: required

The above works fine with DuckDB 0.9.2

I attached the metadata file, which was created by Iceberg. 00006-0d6ca258-8039-446f-843a-69e21344f272.metadata.json

[harel-e]

It seems that if the schema contains a required field ("required" : true) the extension fails. I tried it with https://duckdb.org/data/iceberg_data.zip I changed one of the fields in v2.metadata.json from "required" : false to "required" : true

0.9.2 - No problems

0.10.0

SELECT * FROM iceberg_scan('data/iceberg/lineitem_iceberg', allow_moved_paths = true);
Error: IO Error: Invalid field found while parsing field: required

[Sol-Hee]

I'm experiencing the same problem. I upgraded the code that used to work at 0.9.2 to 0.10.0 and got IOException.

IOException: IO Error: Invalid field found while parsing field: type

[samansmink]

Thanks for reporting @harel-e and @Sol-Hee.

This is a known issue that is a result from https://github.com/duckdb/duckdb_iceberg/pull/30. In that PR we switched from parsing the parquet schemas to actually parsing the iceberg schema. However, since duckdb's schema param for the parquet reader does not yet support nested types, this support was dropped. The way forward here is to:

• add support for nested types in duckdb parquet reader's schema option
• add support for iceberg nested types in this extension

[harel-e]

@samansmink Thank you for this wonderful extension. I wonder if there an estimate when this issue will be fixed and are there open issues on the two items above? I can assist with testing if needed. Thanks

[samansmink]

Hey @harel-e, at this moment we can not give any time estimate on this, sorry!

[harel-e]

Ok, thanks for the update. Too bad I cannot upgrade. I'll stick to DuckDB 0.9.2 until the issue is fixed.

[veca1982]

Hello, first of all thanks to DuckDB community for beautiful work. I stumbled across this issue facing the same problem :). One can shortcut the problem. It assumes working with DuckDB through ccde. We just need to find iceberg data files (in my case parquet files) from table scan (iceberg API) and input those file paths to DuckDB read_parquet function. Snippet would be something like this (using python and pyiceberg api)

table = catalog.load_table("some.table")
scan = table.scan(row_filter=And(GreaterThanOrEqual("ts", "2024-03-14T08:00:00.000+00:00"), LessThanOrEqual("ts", "2024-03-14T08:10:00.000+00:00"))
files_data = scan.plan_files()
scan_file_paths = [file_data.file.file_path for file_data in files_data]

Let say you're using jupyter notebook with jupysql

%%sql
SELECT count(*)
FROM read_parquet({{scan_file_paths}});

[harel-e]

@veca1982 - Thanks for the workaround suggestion. Does this approach handle updates/deletes ?

[veca1982]

No, it works for Inserts/Append only. For updates/deletes you can use pyiceberg table scan, output it to arrow table, pandas dataframe, DuckDb table... And than query it with DuckDB 😁.

[rustyconover]

I think I've addressed this with #50, it should fix parsing the metadata.

[harel-e]

@rustyconover - can't wait to test it out. I hope it will make it to the 0.10.2 release.

[harel-e]

I just tested #50 - It does indeed fix the metadata parsing issue.

[samansmink]

fixed by #50