MB-55053 : Fix UDF functions being loaded into cache every time the f…

…unction is executed

Change-Id: I90d0c042e7c154e282c4a6d5b9a9b7a0a8ba8275
Reviewed-on: https://review.couchbase.org/c/query/+/184782
Reviewed-by: Donald Haggart <donald.haggart@couchbase.com>
Reviewed-by: Marco Greco <marco.greco@couchbase.com>
Tested-by: Dhanya Gowrish <dhanya.gowrish@couchbase.com>

functions/functions.go

@@ -424,7 +424,7 @@ func ExecuteFunction(name FunctionName, modifiers Modifier, values []value.Value
 		// note that since we reload the body outside of a cache lock (not to lock
 		// out the whole cache bucket), there might be some temporary pile up on
 		// storage
-		if name.CheckStorage() {
+		if entry.FunctionName.CheckStorage() {
 			var tag atomic.AlignedInt64
 
 			// reserve a change counter and load new body