This is an automated email from the ASF dual-hosted git repository. ritik pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/asterixdb.git
commit 7b8a94cdb95c3619ea40b2e8a2fe14cca472a89c Author: Michael Blow <[email protected]> AuthorDate: Sat Dec 21 11:16:52 2024 -0500 [NO ISSUE][*DB][MISC] Updates to accommodate Java 21 - Update javaparser plugin for Java 21 - Update azure utils validation to accommodate Java 21 In Java 21, the URL ctor does (additional) validation of the URL than was done previously- update the adapter setup logic to accommodate failures when setting a malformed URL on the client builder Ext-ref: MB-65321 Change-Id: I31ef40ad04861b7d45f661bcc5550687272f4b8e Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19416 Reviewed-by: Hussain Towaileb <[email protected]> Tested-by: Michael Blow <[email protected]> --- .../asterix/external/util/azure/blob_storage/AzureUtils.java | 12 ++++++++++-- asterixdb/pom.xml | 2 +- hyracks-fullstack/pom.xml | 2 +- 3 files changed, 12 insertions(+), 4 deletions(-) diff --git a/asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/azure/blob_storage/AzureUtils.java b/asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/azure/blob_storage/AzureUtils.java index 0dc9ad2807..13480fdce2 100644 --- a/asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/azure/blob_storage/AzureUtils.java +++ b/asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/azure/blob_storage/AzureUtils.java @@ -118,7 +118,11 @@ public class AzureUtils { if (endpoint == null) { throw new CompilationException(PARAMETERS_REQUIRED, ENDPOINT_FIELD_NAME); } - builder.endpoint(endpoint); + try { + builder.endpoint(endpoint); + } catch (Exception ex) { + throw new CompilationException(ErrorCode.EXTERNAL_SOURCE_ERROR, ex, getMessageOrToString(ex)); + } // Shared Key if (accountName != null || accountKey != null) { @@ -271,7 +275,11 @@ public class AzureUtils { if (endpoint == null) { throw new CompilationException(PARAMETERS_REQUIRED, ENDPOINT_FIELD_NAME); } - builder.endpoint(endpoint); + try { + builder.endpoint(endpoint); + } catch (Exception ex) { + throw new CompilationException(ErrorCode.EXTERNAL_SOURCE_ERROR, ex, getMessageOrToString(ex)); + } // Shared Key if (accountName != null || accountKey != null) { diff --git a/asterixdb/pom.xml b/asterixdb/pom.xml index e4d9a2684f..7799ddce40 100644 --- a/asterixdb/pom.xml +++ b/asterixdb/pom.xml @@ -660,7 +660,7 @@ <dependency> <groupId>com.github.javaparser</groupId> <artifactId>javaparser-core</artifactId> - <version>3.25.5</version> + <version>3.26.3</version> </dependency> </dependencies> </plugin> diff --git a/hyracks-fullstack/pom.xml b/hyracks-fullstack/pom.xml index 66b4ebb947..1b65a1bfa1 100644 --- a/hyracks-fullstack/pom.xml +++ b/hyracks-fullstack/pom.xml @@ -775,7 +775,7 @@ <dependency> <groupId>com.github.javaparser</groupId> <artifactId>javaparser-core</artifactId> - <version>3.25.5</version> + <version>3.26.3</version> </dependency> </dependencies> </plugin>
