Devin Smith

Results 190 comments of Devin Smith

https://errorprone.info/ could be useful

https://github.com/autonomousapps/dependency-analysis-gradle-plugin claims to do exactly this! > Dependencies declared on the wrong configuration (api vs implementation vs compileOnly, etc.).

This is mainly about fixing the "why does it hang", and/or improving the error messages. It may also be reasonable to consider cpp client support for `arrow::BinaryArray`

Noting interest here too; there are some guts that I'd liked exposed that make streaming / bao-like consumption more possible.

I can also make an argument that our default should be: 1. If a schema is provided, use that schema 2. Otherwise, use schema `table.schema()` IE, if providing a specific...

The `table.schema()` regime by default also allows the current constructors from `io.deephaven.iceberg.layout.IcebergBaseLayout` (and derived) to still be valid; while we should be _okay_ breaking these, we eventually need to get...

Q: does `PartitionSpec` come into play, and if yes, is it always the case that the schema is equal to `PartitionSpec#schema`?

Note: checking w/ DHE on their usage, which seems to always pass `ParquetInstructions.EMPTY` - they might have this luxury because they control the writing and can assume 100% match between...

Note: the core of this PR has been merged as part of #6184; I do think it makes sense to continue pushing forward and try to be stricter.

Eric has run into this on M1 Mac; disabling containerd worked around the issue. For reference: ``` % docker version Client: Version: 28.3.3 API version: 1.51 Go version: go1.24.5 Git...