Merge pull request #1019 from dolthub/james/uuid

UUID function is no longer based off current time
dolthub · May 18, 2022 · d635025 · d635025
2 parents fbd5e8b + fc7ab64
commit d635025
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/enginetest/queries.go b/enginetest/queries.go
@@ -6797,6 +6797,12 @@ var QueryTests = []QueryTest{
 		Query:    `show function status where Db='mydb'`,
 		Expected: []sql.Row{},
 	},
+	{
+		Query: `select uuid() = uuid()`,
+		Expected: []sql.Row{
+			{false},
+		},
+	},
 }
 
 var KeylessQueries = []QueryTest{

diff --git a/sql/expression/function/uuid.go b/sql/expression/function/uuid.go
@@ -77,12 +77,7 @@ func (u UUIDFunc) Type() sql.Type {
 }
 
 func (u UUIDFunc) Eval(ctx *sql.Context, row sql.Row) (interface{}, error) {
-	nUUID, err := uuid.NewUUID()
-	if err != nil {
-		return nil, err
-	}
-
-	return nUUID.String(), nil
+	return uuid.New().String(), nil
 }
 
 func (u UUIDFunc) WithChildren(children ...sql.Expression) (sql.Expression, error) {