Re: [VOTE] Endpoint for refreshing vended credentials

2024-10-24 Thread Eduard Tudenhöfner
h Jahagirdar <2am...@gmail.com> >> *Gesendet:* Tuesday, October 22, 2024 8:00:00 PM >> *An:* dev@iceberg.apache.org >> *Betreff:* Re: [VOTE] Endpoint for refreshing vended credentials >> >> +1 (binding) >> >> On Tue, Oct 22, 2024 at 5:20 PM rdb...@gmail.

Re: [VOTE] Endpoint for refreshing vended credentials

2024-10-22 Thread Fanng
ff:* Re: [VOTE] Endpoint for refreshing vended credentials > > +1 (binding) > > On Tue, Oct 22, 2024 at 5:20 PM rdb...@gmail.com wrote: > > +1 (binding) > > Thanks for your work on this! > > On Tue, Oct 22, 2024 at 2:47 PM Prashant Singh > wrote: > > +1 (non-

Re: [VOTE] Endpoint for refreshing vended credentials

2024-10-22 Thread Christian Thiel
+1 (non-binding). Great feature, thanks! Von: Amogh Jahagirdar <2am...@gmail.com> Gesendet: Tuesday, October 22, 2024 8:00:00 PM An: dev@iceberg.apache.org Betreff: Re: [VOTE] Endpoint for refreshing vended credentials +1 (binding) On Tue, Oct 22, 202

Re: [VOTE] Endpoint for refreshing vended credentials

2024-10-22 Thread Amogh Jahagirdar
+1 (binding) On Tue, Oct 22, 2024 at 5:20 PM rdb...@gmail.com wrote: > +1 (binding) > > Thanks for your work on this! > > On Tue, Oct 22, 2024 at 2:47 PM Prashant Singh > wrote: > >> +1 (non-binding) >> >> Regards, >> Prashant >> >> On Tue, Oct 22, 2024 at 10:50 AM John Zhuge wrote: >> >>> +1

Re: [VOTE] Endpoint for refreshing vended credentials

2024-10-22 Thread rdb...@gmail.com
+1 (binding) Thanks for your work on this! On Tue, Oct 22, 2024 at 2:47 PM Prashant Singh wrote: > +1 (non-binding) > > Regards, > Prashant > > On Tue, Oct 22, 2024 at 10:50 AM John Zhuge wrote: > >> +1 (non-binding) >> >> John Zhuge >> >> >> On Tue, Oct 22, 2024 at 9:45 AM Jack Ye wrote: >>

Re: [VOTE] Endpoint for refreshing vended credentials

2024-10-22 Thread Prashant Singh
+1 (non-binding) Regards, Prashant On Tue, Oct 22, 2024 at 10:50 AM John Zhuge wrote: > +1 (non-binding) > > John Zhuge > > > On Tue, Oct 22, 2024 at 9:45 AM Jack Ye wrote: > >> +1 (binding) >> >> Best, >> Jack Ye >> >> On Tue, Oct 22, 2024 at 9:32 AM Dmitri Bourlatchkov >> wrote: >> >>> Than

Re: [VOTE] Endpoint for refreshing vended credentials

2024-10-22 Thread John Zhuge
+1 (non-binding) John Zhuge On Tue, Oct 22, 2024 at 9:45 AM Jack Ye wrote: > +1 (binding) > > Best, > Jack Ye > > On Tue, Oct 22, 2024 at 9:32 AM Dmitri Bourlatchkov > wrote: > >> Thanks for the reply Eduard! >> >> I think it is fine to defer fine-tuning credential refreshes to a later >> PR.

Re: [VOTE] Endpoint for refreshing vended credentials

2024-10-22 Thread Jack Ye
+1 (binding) Best, Jack Ye On Tue, Oct 22, 2024 at 9:32 AM Dmitri Bourlatchkov wrote: > Thanks for the reply Eduard! > > I think it is fine to defer fine-tuning credential refreshes to a later PR. > > I'm upgrading my vote to +1 (non-binding). > > Cheers, > Dmitri. > > On Tue, Oct 22, 2024 at 1

Re: [VOTE] Endpoint for refreshing vended credentials

2024-10-22 Thread Dmitri Bourlatchkov
Thanks for the reply Eduard! I think it is fine to defer fine-tuning credential refreshes to a later PR. I'm upgrading my vote to +1 (non-binding). Cheers, Dmitri. On Tue, Oct 22, 2024 at 11:11 AM Eduard Tudenhöfner < etudenhoef...@apache.org> wrote: > Hey Dmitri, > > the idea behind the endpo

Re: [VOTE] Endpoint for refreshing vended credentials

2024-10-22 Thread Jean-Baptiste Onofré
Hi +1 (non binding) @Dmitri I don't think it's an issue: the endpoint can return multiple credentials, the client will take one. The credential refresh is not so costly though (and required to verify valid credential at retrieval time). Regards JB On Tue, Oct 22, 2024 at 5:09 PM Eduard Tudenhöf

Re: [VOTE] Endpoint for refreshing vended credentials

2024-10-22 Thread Eduard Tudenhöfner
Hey Dmitri, the idea behind the endpoint itself is really just to provide *valid* credentials for a given table when a client asks for them. If the server returned you two S3 credentials, the client will use the one with the longest prefix and if that credential expires, it will ask the server aga

Re: [VOTE] Endpoint for refreshing vended credentials

2024-10-21 Thread Dmitri Bourlatchkov
-0 (non-binding) If multiple credentials are vended for a table (which is allowed) the current API requires all credentials to be refreshed, when any of the previous credentials expires. I think this is suboptimal (but can probably be made to work in most practical cases). Cheers, Dmitri. On Mon

Re: [VOTE] Endpoint for refreshing vended credentials

2024-10-21 Thread Daniel Weeks
+1 On Mon, Oct 21, 2024 at 3:07 AM Eduard Tudenhöfner wrote: > Hey everyone, > > I'd like to vote on #11281 , > which introduces a new endpoint and allows > retrieving/refreshing vended credentials for a given table. > > Please vote +1 if you general