From 3b7484f69c403aafedaba4305693ee47d4cbc63f Mon Sep 17 00:00:00 2001 From: Wei Zheng Date: Mon, 22 Aug 2022 09:52:30 -0700 Subject: [PATCH] Make functionNullability property consistent in ResolvedFunction The mismatch of "nullability" and "functionNullability" could cause failure of JSON deserialization for ResolvedFunction --- .../src/main/java/io/trino/metadata/ResolvedFunction.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/core/trino-main/src/main/java/io/trino/metadata/ResolvedFunction.java b/core/trino-main/src/main/java/io/trino/metadata/ResolvedFunction.java index dfdcdc1dfc43..fe5f66708105 100644 --- a/core/trino-main/src/main/java/io/trino/metadata/ResolvedFunction.java +++ b/core/trino-main/src/main/java/io/trino/metadata/ResolvedFunction.java @@ -70,7 +70,7 @@ public ResolvedFunction( @JsonProperty("id") FunctionId functionId, @JsonProperty("functionKind") FunctionKind functionKind, @JsonProperty("deterministic") boolean deterministic, - @JsonProperty("nullability") FunctionNullability functionNullability, + @JsonProperty("functionNullability") FunctionNullability functionNullability, @JsonProperty("typeDependencies") Map typeDependencies, @JsonProperty("functionDependencies") Set functionDependencies) { @@ -78,7 +78,7 @@ public ResolvedFunction( this.functionId = requireNonNull(functionId, "functionId is null"); this.functionKind = requireNonNull(functionKind, "functionKind is null"); this.deterministic = deterministic; - this.functionNullability = requireNonNull(functionNullability, "nullability is null"); + this.functionNullability = requireNonNull(functionNullability, "functionNullability is null"); this.typeDependencies = ImmutableMap.copyOf(requireNonNull(typeDependencies, "typeDependencies is null")); this.functionDependencies = ImmutableSet.copyOf(requireNonNull(functionDependencies, "functionDependencies is null")); checkArgument(functionNullability.getArgumentNullable().size() == signature.getArgumentTypes().size(), "signature and functionNullability must have same argument count");