Timestamp value conversion in Cloud Datastore connector is wrong

[damccorm]

Code in org.apache.beam.sdk.io.gcp.datastore.EntityToRow class:

com.google.protobuf.Timestamp time = val.getTimestampValue();
long millis = time.getSeconds() * 1000
+ time.getNanos() / 1000;
return Instant.ofEpochMilli(millis).toDateTime();

1 millisecond = 1,000,000 nanoseconds

Imported from Jira BEAM-14541. Original Jira may contain additional context.
Reported by: yixiaoshen.

[damccorm]

Unable to assign user @yixiaoshen. If able, self-assign, otherwise tag @damccorm so that he can assign you. Because of GitHub's spam prevention system, your activity is required to enable assignment in this repo.

[yixiaoshen]

@damccorm I don't have permission to assign the issue, can you help assign the issue to me? thanks!

[damccorm]

Done - FWIW once #21719 is merged, you'll be able to self-assign by commenting .take-issue

[yixiaoshen]

This is fixed by #17789, but I don't seem to have permission to .close-issue

[damccorm]

This is fixed by #17789, but I don't seem to have permission to .close-issue

You just did 😃 anyone can do it by commenting .close-issue like you just did