How to convert LocalDateTime to com.google.protobuf.Timestamp?

Question

I have an instance of LocalDateTime, which I get from the repository layer, and I need to convert it to a Timestamp (Protocol Buffer) instance.

I have used to following approach for the conversion:

LocalDateTime localDateTime = LocalDateTime.now();//this can be any date

Instant instant = localDateTime.toInstant(ZoneOffset.UTC);

Timestamp timestamp = Timestamp.newBuilder()
                            .setSeconds(instant.getEpochSecond())
                            .setNanos(instant.getNano())
                            .build();

Is the ZoneOffset used here, to convert localDateTime to an instance of Instant, correct?

I have used the UTC offset because the comment on the "seconds" attribute in the Timestamp class says the following:

Represents seconds of UTC time since Unix epoch 1970-01-01T00:00:00Z. Must be from 0001-01-01T00:00:00Z to 9999-12-31T23:59:59Z inclusive

Have I used the correct ZoneOffset and is my conversion approach correct?

Yes, zone offset is necessary to resolve a `LocalDateTime` to an `Instant`. The rest of your question is unclear. The part after "Because" doesn't seem connected to the part with the question mark. Please clarify. — Misha, Jun 07 '18 at 06:05
We cannot tell you what timezone to use. Why does your repository return a `LocalDateTime` instead of an `Instant`? Hopefully, you can find out what time zone the datetime is relative to. If it's not documented anywhere, the datetime is probably in server-local time (returned by `ZoneId.systemDefault()`). Relying on system timezone is an undesirable practice but people do it all the time nonetheless — Misha, Jun 07 '18 at 07:41

score 6 · Accepted Answer · answered Jun 07 '18 at 07:39

In general, no, your approach is not correct. The reason is that a LocalDateTime does not have an associated timezone, so it is ambiguous by nature. To convert it to an actual timestamp (an absolute point in time, independent of timezones), you need to know what timezone it was measured in.

By calling localDateTime.toInstant(ZoneOffset.UTC), you are assuming that your localDateTime was actually measured in the UTC timezone. Instead, you should be using the timezone that the LocalDateTime is stored in. If you don't know, then your input data is inherently ambiguous and you'll need to fix that first.

Note that this has nothing to do with the fact that the Unix epoch is usually specified in UTC. We might as well say that the Unix epoch is 1970-01-01T08:00:00+08:00, and it would be the same instant in time.

The rest of it seems correct to me.

Thanks for the clarification Thomas! – uneq95 Jun 08 '18 at 05:37 — uneq95, Jun 08 '18 at 05:37

score 3 · Answer 2 · answered Feb 11 '21 at 21:04

Here's a routine that pulls together the comments from the question and actual answer to this question:

 protected Timestamp convertLocalDateTimeToGoogleTimestamp(LocalDateTime localDateTime) {
    Instant instant = localDateTime.toInstant(ZoneOffset.UTC);

    Timestamp result = Timestamp.newBuilder()
            .setSeconds(instant.getEpochSecond())
            .setNanos(instant.getNano())
            .build();

    return result;
  }

score 0 · Answer 3 · answered Jul 12 '22 at 09:53

LocalDateTime localDateTime = LocalDateTime.now();//this can be any date

Instant instant = localDateTime.atZone(ZoneId.systemDefault()).toInstant();

Timestamp timestamp = Timestamp.newBuilder()
                            .setSeconds(instant.getEpochSecond())
                            .setNanos(instant.getNano())
                            .build();

score -1 · Answer 4 · edited Mar 04 '23 at 12:54

-1

Try this:

Timestamp currentTimeStamp =   Timestamps.fromMillis(System.currentTimeMillis());

edited Mar 04 '23 at 12:54

Sunderam Dubey

1
11
20
40

answered Mar 03 '23 at 23:32

Sneha Mule

641
8
6

How to convert LocalDateTime to com.google.protobuf.Timestamp?

4 Answers4