Re: [DISCUSS] Gradle and Maven runner plugins

2025-07-16 Thread Dmitri Bourlatchkov
I was reviewing our Antlr4 dependencies in Spark tests yesterday and realised that having a running plugin would simplify those tests a lot. The current problem is that Quarkus has a different dependency tree from Spark and combining the two in the same test run requires close attention to manag

Concurrent Namespace Update Fails with RuntimeException and 500 Error #1390

2025-07-16 Thread Rizzo Cascio, Fabio
Hi all, While working on Issue #1390, what initially seemed like a quick fix has turned into a more complex problem. Following Dimas-b's suggestion, I'm opening this up for a wider discussion. You can see the comments on my PR #1989

Re: Proposal: Optional clientId / clientSecret in createPrincipal API for Migration Support

2025-07-16 Thread Arun Suri
I’m aligned with the phased approach you’ve outlined: supporting password resets for existing principals now, while steadily enhancing IdP support and making it the default over time I will go ahead with making changes based on the discussions Thank you for the suggestions On Tue, Jul 15, 2025 at

Re: Concurrent Namespace Update Fails with RuntimeException and 500 Error #1390

2025-07-16 Thread Dmitri Bourlatchkov
Hi Fabio! Thanks for starting this thread! Just to clarify my GH comments: I think we need to consider table and namespace updates separately. As far as namespace updates are concerned, my reading of the Iceberg REST Catalog API spec is that the spec does not define anything about conflicts. It

Re: Concurrent Namespace Update Fails with RuntimeException and 500 Error #1390

2025-07-16 Thread Jean-Baptiste Onofré
Hi Fabio I think the update of namespace entities should be specific compared to other entities, because the workflow is slightly different and can be updated "transitively". Overall PR #1989 looks good to me, but I'm not sure throwing CommitException is the best option here, I would prefer to spe