Thanks Ashok for working on this! Sorry for the late reply. Will review it
over the weekend.

Yufei


On Thu, Jun 26, 2025 at 5:02 AM Ashok Borra <ashok.bo...@fivetran.com>
wrote:

> Hi Yufei,
>
> Sorry for the delay. I added support for feature flag for a safe rollout.
>
> Could you please review the PR
> <https://github.com/apache/polaris/pull/1424> when you get some time?
>
> Thanks,
> Ashok
>
> On Thu, May 29, 2025 at 12:12 PM Ashok Borra <ashok.bo...@fivetran.com>
> wrote:
>
>> +Raajan Ashish Pal <raajanashish....@fivetran.com>
>>
>> On Thu, May 29, 2025 at 4:33 AM Yufei Gu <flyrain...@gmail.com> wrote:
>>
>>> Hi Ashok,
>>>
>>> It makes sense to separate both options into different PRs. We will need
>>> a feature flag to control them though, see more details in class
>>> FeatureConfiguration. The potential options would be:
>>> 1. No KMS support
>>> 2. KMS support with Approach #3
>>> 3. KMS support with Approach #4
>>>
>>> Yufei
>>>
>>>
>>> On Mon, May 26, 2025 at 2:09 AM Jean-Baptiste Onofré <j...@nanthrax.net>
>>> wrote:
>>>
>>>> Hi Ashok
>>>>
>>>> Thanks for the update. I will take a look as I'm back from several
>>>> trips :)
>>>>
>>>> Regards
>>>> JB
>>>>
>>>> On Fri, May 23, 2025 at 5:10 PM Ashok Krishna <ashok.rex.2...@gmail.com>
>>>> wrote:
>>>> >
>>>> > Hi folks,
>>>> >
>>>> >
>>>> > This is regarding the addition of KMS support for AWS S3 in Polaris.
>>>> > Several approaches were proposed in the PR
>>>> > <https://github.com/apache/polaris/pull/1424>, and based on our last
>>>> > community sync, we aligned on the following:
>>>> >
>>>> >
>>>> >
>>>> >    - Using an IAM policy with *Resource: '*'* at the catalog level
>>>> >    (Approach #4)
>>>> >
>>>> >
>>>> >    - Using table-level keys as per the Iceberg spec (Approach #3)
>>>> >
>>>> >
>>>> > The current PR <https://github.com/apache/polaris/pull/1424>
>>>> implements
>>>> > Approach #4 at the catalog level. I’d like to confirm whether it’s
>>>> okay to
>>>> > split the implementation and submit Approach #3 as a separate PR.
>>>> >
>>>> > PR - https://github.com/apache/polaris/pull/1424
>>>>
>>>
>>
>> --
>> Ashok Borra
>>
>> Staff Software Engineer
>>
>> Engineering Department | Fivetran
>>
>> ashok.bo...@fivetran.com
>> fivetran.com <//fivetran.com>
>> 80, Indiqube Lakeside, 3rd Floor, 2, Outer Ring Rd, Green Glen Layout,
>> Bengaluru, KA 560103 IN
>> <http://www.fivetran.com>
>> [image: youtube] <https://www.youtube.com/c/Fivetran> [image: twitter]
>> <https://twitter.com/fivetran?ref_src=twsrc%5Egoogle%7Ctwcamp%5Eserp%7Ctwgr%5Eauthor>
>>  [image:
>> linkedin] <https://www.linkedin.com/company/fivetran>
>>
>
>
> --
> Ashok Borra
>
> Staff Software Engineer
>
> Engineering Department | Fivetran
>
> ashok.bo...@fivetran.com
> fivetran.com <//fivetran.com>
> 80, Indiqube Lakeside, 3rd Floor, 2, Outer Ring Rd, Green Glen Layout,
> Bengaluru, KA 560103 IN
> <http://www.fivetran.com>
> [image: youtube] <https://www.youtube.com/c/Fivetran> [image: twitter]
> <https://twitter.com/fivetran?ref_src=twsrc%5Egoogle%7Ctwcamp%5Eserp%7Ctwgr%5Eauthor>
>  [image:
> linkedin] <https://www.linkedin.com/company/fivetran>
>

Reply via email to