Browse Source

FIxed syntaxis mistake unixEpochNanoFrom and unixEpochNanoTo

Ander Arguiñano 7 years ago
parent
commit
3801a75131

+ 2 - 2
pkg/tsdb/mssql/macros.go

@@ -102,9 +102,9 @@ func (m *msSqlMacroEngine) evaluateMacro(name string, args []string) (string, er
 		}
 		return fmt.Sprintf("%s >= %d AND %s <= %d", args[0], m.timeRange.GetFromAsTimeUTC().UnixNano(), args[0], m.timeRange.GetToAsTimeUTC().UnixNano()), nil
 	case "__unixEpochNanoFrom":
-		return fmt.Sprintf("'%d'", m.timeRange.GetFromAsTimeUTC().UnixNano()), nil
+		return fmt.Sprintf("%d", m.timeRange.GetFromAsTimeUTC().UnixNano()), nil
 	case "__unixEpochNanoTo":
-		return fmt.Sprintf("'%d'", m.timeRange.GetToAsTimeUTC().UnixNano()), nil
+		return fmt.Sprintf("%d", m.timeRange.GetToAsTimeUTC().UnixNano()), nil
 	case "__unixEpochGroup":
 		if len(args) < 2 {
 			return "", fmt.Errorf("macro %v needs time column and interval and optional fill value", name)

+ 2 - 2
pkg/tsdb/mssql/macros_test.go

@@ -142,14 +142,14 @@ func TestMacroEngine(t *testing.T) {
 				sql, err := engine.Interpolate(query, timeRange, "select $__unixEpochNanoFrom()")
 				So(err, ShouldBeNil)
 
-				So(sql, ShouldEqual, fmt.Sprintf("select '%d'", from.UnixNano()))
+				So(sql, ShouldEqual, fmt.Sprintf("select %d", from.UnixNano()))
 			})
 
 			Convey("interpolate __unixEpochNanoTo function", func() {
 				sql, err := engine.Interpolate(query, timeRange, "select $__unixEpochNanoTo()")
 				So(err, ShouldBeNil)
 
-				So(sql, ShouldEqual, fmt.Sprintf("select '%d'", to.UnixNano()))
+				So(sql, ShouldEqual, fmt.Sprintf("select %d", to.UnixNano()))
 			})
 
 			Convey("interpolate __unixEpochGroup function", func() {

+ 2 - 2
pkg/tsdb/postgres/macros.go

@@ -128,9 +128,9 @@ func (m *postgresMacroEngine) evaluateMacro(name string, args []string) (string,
 		}
 		return fmt.Sprintf("%s >= %d AND %s <= %d", args[0], m.timeRange.GetFromAsTimeUTC().UnixNano(), args[0], m.timeRange.GetToAsTimeUTC().UnixNano()), nil
 	case "__unixEpochNanoFrom":
-		return fmt.Sprintf("'%d'", m.timeRange.GetFromAsTimeUTC().UnixNano()), nil
+		return fmt.Sprintf("%d", m.timeRange.GetFromAsTimeUTC().UnixNano()), nil
 	case "__unixEpochNanoTo":
-		return fmt.Sprintf("'%d'", m.timeRange.GetToAsTimeUTC().UnixNano()), nil
+		return fmt.Sprintf("%d", m.timeRange.GetToAsTimeUTC().UnixNano()), nil
 	case "__unixEpochGroup":
 		if len(args) < 2 {
 			return "", fmt.Errorf("macro %v needs time column and interval and optional fill value", name)

+ 2 - 2
pkg/tsdb/postgres/macros_test.go

@@ -125,14 +125,14 @@ func TestMacroEngine(t *testing.T) {
 				sql, err := engine.Interpolate(query, timeRange, "select $__unixEpochNanoFrom()")
 				So(err, ShouldBeNil)
 
-				So(sql, ShouldEqual, fmt.Sprintf("select '%d'", from.UnixNano()))
+				So(sql, ShouldEqual, fmt.Sprintf("select %d", from.UnixNano()))
 			})
 
 			Convey("interpolate __unixEpochNanoTo function", func() {
 				sql, err := engine.Interpolate(query, timeRange, "select $__unixEpochNanoTo()")
 				So(err, ShouldBeNil)
 
-				So(sql, ShouldEqual, fmt.Sprintf("select '%d'", to.UnixNano()))
+				So(sql, ShouldEqual, fmt.Sprintf("select %d", to.UnixNano()))
 			})
 
 			Convey("interpolate __unixEpochGroup function", func() {