ArtifactVersion.CreateTime as float64 is ambiguous and non-idiomatic

[w1t]

attrs.Created is a time.Time, but this converts it to float64(attrs.Created.Unix()) for ArtifactVersion.CreateTime. That drops sub-second precision, and the bigger issue is that CreateTime probably shouldn’t be a float64 in the first place. time.Time would be more idiomatic, or int64 if this is intended to store a Unix timestamp.

adk-go/artifact/gcsartifact/service.go

Line 403 in b8eb8c5

CreateTime: float64(attrs.Created.Unix()),

ArtifactVersion: &artifact.ArtifactVersion{
			Version:        version,
			CanonicalURI:   canonicalURI,
			CustomMetadata: customMeta,
			CreateTime:     float64(attrs.Created.Unix()), 
			MimeType:       attrs.ContentType,
		},

https://github.com/googleapis/google-cloud-go/blob/0b388a07dc2c2fcba5dbfca5a5641c38d1dc8067/storage/storage.go#L1612

[indurireddy-TF]

I was able to reproduce this locally sub second precision is lost during conversion. This also makes the field slightly ambiguous for consumers. Switching to time.Time or int64 would make it more precise and idiomatic.