Imported ML model robustness

- Handle properly that compiling an expression leads to adding new expressions
- Don't load rank profile models as global when reading stored models
- Proper verification of model pseudofeature arguments before taking an action
- Handle missing models and missing models in existing paths properly
- Don't require return types to be resolved for all functions yet
  (they won't be for those added during comppilation)

1 files changed, 2 insertions, 2 deletions

diff --git a/indexinglanguage/src/test/java/com/yahoo/vespa/indexinglanguage/expressions/GetFieldTestCase.java b/indexinglanguage/src/test/java/com/yahoo/vespa/indexinglanguage/expressions/GetFieldTestCase.java
index 3b5569995e6..af1a58a1de6 100644
--- a/indexinglanguage/src/test/java/com/yahoo/vespa/indexinglanguage/expressions/GetFieldTestCase.java
+++ b/indexinglanguage/src/test/java/com/yahoo/vespa/indexinglanguage/expressions/GetFieldTestCase.java
@@ -40,7 +40,7 @@ public class GetFieldTestCase {
         assertVerify(type, exp, DataType.STRING);
         assertVerifyThrows(null, exp, "Expected any input, got null.");
         assertVerifyThrows(DataType.INT, exp, "Expected structured input, got int.");
-        assertVerifyThrows(type, new GetFieldExpression("bar"), "Field 'bar' not found.");
+        assertVerifyThrows(type, new GetFieldExpression("bar"), "Field 'bar' not found in struct type 'my_struct'");
     }
 
     @Test
@@ -79,7 +79,7 @@ public class GetFieldTestCase {
             new GetFieldExpression("foo").execute(new StructDataType("my_struct").createFieldValue());
             fail();
         } catch (IllegalArgumentException e) {
-            assertEquals("Field 'foo' not found.", e.getMessage());
+            assertEquals("Field 'foo' not found in struct type 'my_struct'", e.getMessage());
         }
     }
 }