Re: [PR] Enable HTTP proxy support for the client used by REST Catalog [iceberg]

2025-05-06 Thread via GitHub
flyrain merged PR #12406: URL: https://github.com/apache/iceberg/pull/12406 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@iceberg.a

Re: [PR] Enable HTTP proxy support for the client used by REST Catalog [iceberg]

2025-05-06 Thread via GitHub
sfc-gh-mbaron commented on PR #12406: URL: https://github.com/apache/iceberg/pull/12406#issuecomment-2854848305 @flyrain @adutra @nastra @danielcweeks @amogh-jahagirdar any update here?? -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

Re: [PR] Enable HTTP proxy support for the client used by REST Catalog [iceberg]

2025-05-02 Thread via GitHub
flyrain commented on PR #12406: URL: https://github.com/apache/iceberg/pull/12406#issuecomment-2848156238 Thanks for the discussion. I'm leaning toward keeping proxy separate from `authManager`. The tiny bit we’re adding(an HTTP proxy) is more on the transport-level wiring for the HT

Re: [PR] Enable HTTP proxy support for the client used by REST Catalog [iceberg]

2025-04-28 Thread via GitHub
akhilputhiry commented on PR #12406: URL: https://github.com/apache/iceberg/pull/12406#issuecomment-2837363721 Wanted to follow up on this @amogh-jahagirdar @danielcweeks @nastra @adutra -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] Enable HTTP proxy support for the client used by REST Catalog [iceberg]

2025-04-06 Thread via GitHub
akhilputhiry commented on PR #12406: URL: https://github.com/apache/iceberg/pull/12406#issuecomment-2781514674 @sfc-gh-mbaron I am also eagerly waiting for this to be merged @nastra @adutra @danielcweeks @amogh-jahagirdar Could you folks please help to move this forward. Thank

Re: [PR] Enable HTTP proxy support for the client used by REST Catalog [iceberg]

2025-04-06 Thread via GitHub
sfc-gh-mbaron commented on PR #12406: URL: https://github.com/apache/iceberg/pull/12406#issuecomment-2781483956 @akhilputhiry @danielcweeks @adutra any plans to merge this soon? This is blocking some things I'm trying to do and trying to get a sense of timing. -- This is an automated mess

Re: [PR] Enable HTTP proxy support for the client used by REST Catalog [iceberg]

2025-04-05 Thread via GitHub
danielcweeks commented on PR #12406: URL: https://github.com/apache/iceberg/pull/12406#issuecomment-2744141255 > To summarize my POV: I'm +1 on this PR, provided that we introduce multi-host proxy settings later on. I'm not convinced we want to go down this path until there are real w

Re: [PR] Enable HTTP proxy support for the client used by REST Catalog [iceberg]

2025-03-24 Thread via GitHub
akhilputhiry commented on PR #12406: URL: https://github.com/apache/iceberg/pull/12406#issuecomment-2750174836 @danielcweeks @adutra I believe the current discussion on multiple proxies can be continued in the follow up PR or Slack Do we have agreement on the current proxy enablement?

Re: [PR] Enable HTTP proxy support for the client used by REST Catalog [iceberg]

2025-03-24 Thread via GitHub
adutra commented on PR #12406: URL: https://github.com/apache/iceberg/pull/12406#issuecomment-2742711559 > I have a few concerns here and it may overlap a little with what @adutra was getting at. This appears to tunnel a separate config/auth to the http client as opposed to extending and us

Re: [PR] Enable HTTP proxy support for the client used by REST Catalog [iceberg]

2025-03-20 Thread via GitHub
danielcweeks commented on PR #12406: URL: https://github.com/apache/iceberg/pull/12406#issuecomment-2741090134 I have a few concerns here and it may overlap a little with what @adutra was getting at. This appears to tunnel a separate config/auth to the http client as opposed to extending a