Skip to content

Commit

Permalink
Update exception hanlding
Browse files Browse the repository at this point in the history
Signed-off-by: Andy Kwok <[email protected]>
  • Loading branch information
andy-k-improving committed Feb 7, 2025
1 parent ca79ffc commit 116bda7
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions core/src/main/java/org/opensearch/sql/utils/JsonUtils.java
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@
import com.jayway.jsonpath.Configuration;
import com.jayway.jsonpath.DocumentContext;
import com.jayway.jsonpath.InvalidJsonException;
import com.jayway.jsonpath.InvalidModificationException;
import com.jayway.jsonpath.InvalidPathException;
import com.jayway.jsonpath.JsonPath;
import com.jayway.jsonpath.Option;
Expand Down Expand Up @@ -167,11 +166,6 @@ public static ExprValue setJson(ExprValue json, ExprValue path, ExprValue valueT
} catch (InvalidJsonException e) {
final String errorFormat = "JSON object '%s' is not valid. Error details: %s";
throw new SemanticCheckException(String.format(errorFormat, json, e.getMessage()), e);

} catch (InvalidModificationException
| IllegalArgumentException
| UnsupportedOperationException ex) {
return LITERAL_NULL;
}
}

Expand Down

0 comments on commit 116bda7

Please sign in to comment.