> wrt ISO 8601 timestamps: I'd like to keep things consistent with other
places in the spec, which are typically defined as millisecond values.

Fair enough. Now that the spec states the reference point in time, using
millisecond offsets is fine.

Cheers,
Dmitri.

On Tue, Sep 24, 2024 at 10:41 AM Eduard Tudenhöfner <
etudenhoef...@apache.org> wrote:

> Thanks Dmitri for reviewing the PR and the doc.
>
> wrt ISO 8601 timestamps: I'd like to keep things consistent with other
> places in the spec, which are typically defined as millisecond values.
>
> Thanks
> Eduard
>
> On Fri, Sep 20, 2024 at 4:46 PM Dmitri Bourlatchkov
> <dmitri.bourlatch...@dremio.com.invalid> wrote:
>
>> Thanks for proposing this improvement, Eduard!
>>
>> Overall it seems pretty reasonable to me. I added a few comments in GH
>> and in the doc.
>>
>> One higher level point I'd like to discuss is using ISO 8601 to format
>> expiry timestamps (as opposed to numeric milliseconds values). This should
>> hopefully make the config more human-readable without adding too much
>> processing burden. I hope the standard is well supported by most language
>> libraries now. It is certainly supported by java. WDYT?
>>
>> Thanks,
>> Dmitri.
>>
>> On Fri, Sep 13, 2024 at 12:13 PM Eduard Tudenhöfner <
>> etudenhoef...@apache.org> wrote:
>>
>>> Hey everyone,
>>>
>>> I'd like to propose standardizing the vended credentials used in
>>> loadTable / loadView responses.
>>>
>>> I opened #11118 <https://github.com/apache/iceberg/issues/11118> to
>>> track the proposal in GH.
>>> Please find the proposal doc here
>>> <https://docs.google.com/document/d/1lySd_5hMZNtISLKsOvAq7xiNzdXU6TAoHF_yrOXWQvM/edit?usp=sharing>
>>>  (estimated
>>> read time: 5 minutes).
>>> The proposal requires a spec change, which can be seen in #10722
>>> <https://github.com/apache/iceberg/pull/10722>.
>>>
>>> Thanks,
>>> Eduard
>>>
>>

Reply via email to