Wrap `jiff::Timestamp` and related logics under a wrapper `Timestamp` type

[tisonkun]

Currently, we have several spare parts handling jiff::Timestamp and related datetime logics.

Some of them are located at jiff_util.rs while others are inlined, so you can search jiff:: to find them.

To handle datetime logic modular and centralized, we can follow how reqsign encapsulates it:

• https://github.com/apache/opendal-reqsign/blob/60d1ca93/core/src/time.rs
• refactor: chrono to jiff opendal-reqsign#634

This should be easy to follow so I mark it as a good first issue. Feel free to take it over and submit a patch for review.

[dami0806]

Hello! I'd like to take this issue.
If no one is currently assigned, may I start working on a PR for it?

[tisonkun]

@dami0806 sure! Go for it and feel free to ping me as a reviewer or for advice.

[Xuanwo]

I'm a bit hesitant about this since I don't want to maintain the entire public API surface of jiff. What does our wrapper Timestamp API look like?

[tisonkun]

I don't want to maintain the entire public API surface of jiff

We don't do that. It's basically the same as current state but centralize the date time logics into the wrapper type. Otherwise we may specific calls here and there randomly.

What does our wrapper Timestamp API look like

Should generally like what reqsign does, and it won't be a public API since all of them are now under opendal::raw which does not promise that.

[Xuanwo]

Thank you for explaining.

So will it live in opendal::Metadata? What's the return type of Metadata::last_modified() now?

[tisonkun]

Previously it's chrono::DateTime and now it's jiff:Timestamp. We can unwrap the type as jiff::Timestamp on returning.

[Xuanwo]

Previously it's chrono::DateTime and now it's jiff:Timestamp. We can unwrap the type as jiff::Timestamp on returning.

Seems nice, thank you!

[dami0806]

Hi Xuanwo and tisonkun,

Thanks to your discussion, I’d like to share my understanding and current progress.
As I understand it, the idea behind issue #6647 is as follows,
and I’m proceeding accordingly:

• Introduce a Timestamp wrapper to centralize all date- and time-related logic
  (following a pattern similar to how reqsign wraps chrono).
• Place the wrapper under the opendal::raw module for internal use only, without exposing it in the public API.

Current progress:

• Key groundwork completed: core/src/raw/jiff_util.rs has been removed.
• Implementation completed: core logic of the Timestamp wrapper in core/src/raw/time.rs is finished.
• Ongoing: replacing remaining jiff::Timestamp usages across the codebase and resolving compilation errors.

[tisonkun]

@dami0806 Cool. Looking forward to your (draft) PR :D