Re: how to test row level delete

2021-04-08 Thread Chen Song
Thanks for the clarification. On Tue, Apr 6, 2021 at 10:25 PM OpenInx wrote: > Hi Chen Song > > If want to test the format v2 under your env, you could follow this > comment https://github.com/apache/iceberg/pull/2410#issuecomment-812463051 > to upgrade your iceberg table to format v2. > > The

Re: how to test row level delete

2021-04-06 Thread OpenInx
Hi Chen Song If want to test the format v2 under your env, you could follow this comment https://github.com/apache/iceberg/pull/2410#issuecomment-812463051 to upgrade your iceberg table to format v2. The TableProperties.FORMAT_VERSION was introduced in a separate PoC PR , so we could not find th

Re: how to test row level delete

2021-04-06 Thread Chen Song
Hey I want to quickly follow up on this thread. I cannot seem to find any pull request to expose V2 format version on table creation, specifically for the line below referenced in your previous email. TableProperties.FORMAT_VERSION Can you suggest? I want to create a V2 table to test some row le

Re: how to test row level delete

2020-12-27 Thread OpenInx
> you can apply this patch in your own repository The patch is : https://github.com/apache/iceberg/pull/1978 On Mon, Dec 28, 2020 at 10:32 AM OpenInx wrote: > Hi liubo07199 > > Thanks for testing the iceberg row-level delete, I skimmed the code, it > seems you were trying the equality-delete f

Re: how to test row level delete

2020-12-27 Thread OpenInx
Hi liubo07199 Thanks for testing the iceberg row-level delete, I skimmed the code, it seems you were trying the equality-delete feature. For iceberg users, I think we don't have to write those iceberg internal codes to get this work, this isn't friendly for users. Instead, we usually use the e

how to test row level delete

2020-12-26 Thread 1
Hi, all: I want to try row level delete, but get the exception : IllegalArgumentException: Cannot write delete files in a v1 table. I look over https://iceberg.apache.org/spec/#table-metadata for format-version, it said that An integer version number for the format. Currently, this is