dan-s1 commented on code in PR #10569:
URL: https://github.com/apache/nifi/pull/10569#discussion_r2567097291


##########
nifi-extension-bundles/nifi-snowflake-bundle/nifi-snowflake-services/src/main/java/org/apache/nifi/snowflake/service/StandardSnowflakeIngestManagerProviderService.java:
##########
@@ -58,7 +58,7 @@ public class StandardSnowflakeIngestManagerProviderService 
extends AbstractContr
     public static final PropertyDescriptor HOST_URL = new 
PropertyDescriptor.Builder()
             .name("Snowflake URL")
             .description("Example host url: 
[account-locator].[cloud-region].[cloud]" + 
ConnectionUrlFormat.SNOWFLAKE_HOST_SUFFIX)
-            .addValidator(StandardValidators.NON_BLANK_VALIDATOR)
+            .addValidator(StandardValidators.URL_VALIDATOR)

Review Comment:
   Just to clarify that means the `.dependsOn(ACCOUNT_IDENTIFIER_FORMAT, 
AccountIdentifierFormat.FULL_URL)` for that property is not really a URL?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to