[GitHub] [doris] qidaye commented on a diff in pull request #24309: [fix](multi-catalog)Es catalog needs to verify whether it is a valid configuration.

[via GitHub]

qidaye commented on code in PR #24309:
URL: https://github.com/apache/doris/pull/24309#discussion_r1326875205


##########
fe/fe-core/src/main/java/org/apache/doris/datasource/EsExternalCatalog.java:
##########
@@ -118,6 +119,9 @@ public boolean enableLikePushDown() {
     @Override
     protected void initLocalObjectsImpl() {
         esRestClient = new EsRestClient(getNodes(), getUsername(), 
getPassword(), enableSsl());
+        if (!esRestClient.health()) {
+            throw new DorisEsException("Failed to connect to elasticsearch 
cluster");

Review Comment:
   ```suggestion
               throw new DorisEsException("Failed to connect to ES cluster, 
please check your ES cluster or your ES catalog configuration.");
   ```



-- 
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: commits-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org