🟢 use time tied to local

moving to UTC has a ton of advantages (eg because we do not store locations for time, only unix timestamps), but it comes with a lot of pitfalls (like printing it correctly). Drop UTC in favor of Local time

Signed-off-by: Dominik Richter <[email protected]>

llx/primitives.go

@@ -93,10 +93,10 @@ func TimePrimitive(t *time.Time) *Primitive {
 }
 
 // NeverFutureTime is an indicator for what we consider infinity when looking at time
-var NeverFutureTime = time.Unix(1<<63-1, 0).UTC()
+var NeverFutureTime = time.Unix(1<<63-1, 0)
 
 // NeverPastTime is an indicator for what we consider negative infinity when looking at time
-var NeverPastTime = time.Unix(-(1<<63 - 1), 0).UTC()
+var NeverPastTime = time.Unix(-(1<<63 - 1), 0)
 
 // NeverFuturePrimitive is the special time primitive for the infinite future time
 var NeverFuturePrimitive = TimePrimitive(&NeverFutureTime)

llx/rawdata.go

@@ -58,7 +58,7 @@ func (r *RawData) UnmarshalJSON(data []byte) error {
 			} else if tv < (-(1 << 53)) {
 				r.Value = &NeverPastTime
 			} else {
-				v := time.Unix(int64(tv), 0).UTC()
+				v := time.Unix(int64(tv), 0)
 				r.Value = &v
 			}
 		}

llx/rawdata_test.go

@@ -189,7 +189,7 @@ func TestRawData_JSON(t *testing.T) {
 		StringData("b"),
 		RegexData(""),
 		RegexData("r"),
-		TimeData(time.Time{}),
+		TimeData(time.Time{}.In(time.Local)),
 		TimeData(NeverFutureTime),
 		TimeData(NeverPastTime),
 		// TODO: the raw comparison here does not come out right, because of nano time