Jackyjfhu

Results 6 comments of Jackyjfhu

[KYUUBI #4474][AUTHZ] Support permission control for UPDATE statement with Iceberg 0.12 or below

> I'm wondering how we miss this case of updating iceberg table, as we do have update iceberg table in unit test. Can you provide more information about the issue...

[KYUUBI #4474][AUTHZ] Support permission control for UPDATE statement with Iceberg 0.12 or below

> Seems Iceberg implements update as `ReplaceData` in 0.12 and below, and `UpdateIcebergTable` in 0.13 and above. That's the main reason for missing this case in unit tests as Iceberg...

[KYUUBI #4474][AUTHZ] Support permission control for UPDATE statement with Iceberg 0.12 or below

> Btw, you'd better update `org.apache.kyuubi.plugin.spark.authz.gen.TableCommands` and regenerate `table_command_spec.json` by running `JsonSpecFileGenerator`. OK，thanks for your advice

[KYUUBI #4474][AUTHZ] Support permission control for UPDATE statement with Iceberg 0.12 or below

> A test case shall be added to verify > A test case shall be added to verify How to write test cases? The previous iceberg table update test case...

[KYUUBI #4474][AUTHZ] Support permission control for UPDATE statement with Iceberg 0.12 or below

> > How to write test cases? The previous iceberg table update test case already exists > > the test cases > > * `[KYUUBI #3515] UPDATE TABLE` > *...

[KYUUBI #4474][AUTHZ] Support permission control for UPDATE statement with Iceberg 0.12 or below

> merge master to your PR branch > #4493 is already merged to master. You could merge master to your PR branch or rebase your changes to master. I already...