Another commit to resolve comments

Signed-off-by: Heemin Kim <[email protected]>
opensearch-project · Apr 26, 2023 · ad807fd · ad807fd
1 parent d98f20f
commit ad807fd
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/src/main/java/org/opensearch/geospatial/ip2geo/common/GeoIpDataHelper.java b/src/main/java/org/opensearch/geospatial/ip2geo/common/GeoIpDataHelper.java
@@ -77,7 +77,6 @@ public static void createIndexIfNotExists(
         final TimeValue timeout
     ) {
         if (clusterService.state().metadata().hasIndex(indexName) == true) {
-            log.info("Index {} already exist", indexName);
             return;
         }
         final Map<String, Object> indexSettings = new HashMap<>();
@@ -136,7 +135,7 @@ public static CSVParser getDatabaseReader(final DatasourceManifest manifest) {
                     return new CSVParser(new BufferedReader(new InputStreamReader(zipIn)), CSVFormat.RFC4180);
                 }
             } catch (IOException e) {
-                throw new RuntimeException(e);
+                throw new OpenSearchException("failed to read geoip data from {}", manifest.getUrl(), e);
             }
             throw new OpenSearchException(
                 "database file [{}] does not exist in the zip file [{}]",

diff --git a/src/yamlRestTest/resources/rest-api-spec/test/10_basic.yml b/src/yamlRestTest/resources/rest-api-spec/test/10_basic.yml
@@ -1,4 +1,4 @@
-"Test that geospatial plugin is loaded in OpenSearch":
+"Test that geospatial and job scheduler plugins are loaded in OpenSearch":
   - do:
       cat.plugins:
         local: true