Fokko Driesprong

Results 412 comments of Fokko Driesprong

@tanmayrauth Indeed, as @kevinjqliu is saying, this is staged for the open source sprints for PyData Amsterdam. After today, I'll delete the label, and everyone is welcome to jump in...

I've removed the label and this one is up for grabs. Keep in min that this issue requires `BooleanExpression` to extend `IcebergBaseModel`, so it would be good to get the...

I think it makes sense to drop 3.8 as well :)

I agree that it would be more appropriate to return a `FixedSizeBinary`, let's ask @wgtmac, since he's the expert on Arrow. Just to make sure that there is no historical...

I'm unsure if we want to sort the thing. The most important stuff is now all the way at the bottom :/

@kevinjqliu Let's get https://github.com/apache/iceberg-python/pull/2704 in first as well to avoid having all the exclusions to the top of the file 👍

Hey there, I think the API should be invoked as: ```python with table.update_schema() as updater: updater.add_column(("address", "street"), StringType(), required=False), ... ) ``` However, after a quick test, this still causes...

Agree with @sungwy that this is mostly a documentation issue, so let's extend the docs so ChatGPT can give better answers. Another solution would be: ```python ts_table = catalog.create_table_if_not_exists( 'default.time_series',...

I'm comfortable with having this patch in the read path, but it looks like we're also implicitly adding this to the write path: https://github.com/apache/iceberg-python/blob/e759044601feaf34c99b5c154322b8b52e4f6a30/pyiceberg/io/pyarrow.py#L2576 I'm reluctant to add it there...

If we need to convert from `uuid` to `fixed[16]` to fix this, I think that would be reasonable.