ast: Only realizing lazyObj when Compare() is called on other obj…

…ect type (#6060) This PR pulls out only the performance fix to lazyObj from #6040. Signed-off-by: Johan Fylling <[email protected]>
open-policy-agent · Jun 29, 2023 · 86ec33e · 86ec33e
1 parent 511018e
commit 86ec33e
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/ast/term.go b/ast/term.go
@@ -1861,6 +1861,13 @@ func (l *lazyObj) force() Object {
 }
 
 func (l *lazyObj) Compare(other Value) int {
+	o1 := sortOrder(l)
+	o2 := sortOrder(other)
+	if o1 < o2 {
+		return -1
+	} else if o2 < o1 {
+		return 1
+	}
 	return l.force().Compare(other)
 }