Change start/end time to non-nullable (#109)

## Which problem is this PR solving?
- Part of jaegertracing/jaeger#4150

## Description of the changes
- Make time values non-nullable to avoid pointers

## How was this change tested?
- In jaegertracing/jaeger#6218

---------

Signed-off-by: Yuri Shkuro <[email protected]>

Makefile

@@ -1,5 +1,7 @@
+# Copyright (c) 2023 The Jaeger Authors.
+# SPDX-License-Identifier: Apache-2.0
 
-THRIFT_VER?=0.13
+THRIFT_VER?=0.19
 THRIFT_IMG?=jaegertracing/thrift:$(THRIFT_VER)
 THRIFT=docker run --rm -u $(shell id -u) -v "${PWD}:/data" $(THRIFT_IMG) thrift
 

proto/api_v2/query.proto

@@ -42,11 +42,13 @@ message GetTraceRequest {
   ];
   // Optional. The start time to search trace ID.
   google.protobuf.Timestamp start_time = 2 [
-    (gogoproto.stdtime) = true
+    (gogoproto.stdtime) = true,
+    (gogoproto.nullable) = false
   ];
   // Optional. The end time to search trace ID.
   google.protobuf.Timestamp end_time = 3 [
-    (gogoproto.stdtime) = true
+    (gogoproto.stdtime) = true,
+    (gogoproto.nullable) = false
   ];
 }
 

proto/api_v3/query_service.proto

@@ -30,11 +30,13 @@ message GetTraceRequest {
   string trace_id = 1;
   // Optional. The start time to search trace ID.
   google.protobuf.Timestamp start_time = 2 [
-    (gogoproto.stdtime) = true
+    (gogoproto.stdtime) = true,
+    (gogoproto.nullable) = false
   ];
   // Optional. The end time to search trace ID.
   google.protobuf.Timestamp end_time = 3 [
-    (gogoproto.stdtime) = true
+    (gogoproto.stdtime) = true,
+    (gogoproto.nullable) = false
   ];
 }