Skip to content

Commit 4e3ee64

Browse files
author
Thomas Loubrieu
committed
remove useless comment, add info log for users
1 parent e8b7705 commit 4e3ee64

2 files changed

Lines changed: 5 additions & 6 deletions

File tree

src/main/java/gov/nasa/pds/registry/common/connection/aws/SearchImpl.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -91,13 +91,12 @@ public Search buildListLdds(String namespace) {
9191
return this;
9292
}
9393

94-
@Override
95-
public Search buildListLddsNoCache(String namespace) {
94+
@Override
95+
public Search buildListLddsNoCache(String namespace) {
9696
BoolQuery.Builder journeyman = new BoolQuery.Builder()
9797
.must(this.matchQuery("class_ns", "registry").build(),
9898
this.matchQuery("class_name", "LDD_Info").build(),
9999
this.matchQuery("attr_ns", namespace).build());
100-
// .mustNot(this.matchQuery("xxxxxxxxxnotexisting", String.valueOf(rand.nextInt())).build());
101100
this.craftsman.query(new Query.Builder().bool(journeyman.build()).build());
102101
this.craftsman.requestCache(false);
103102
this.craftsman.size(1000); // have no idea why hardcoded but it is (.es.JsonHelper:265

src/main/java/gov/nasa/pds/registry/common/es/service/JsonLddLoader.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -142,8 +142,7 @@ public void loadOnly(File lddFile, String lddFileName, String namespace, Instant
142142
log.warn("The new LDD {} is not indexed after {} seconds. It may be indexed later, but there may be a delay in loading other LDDs for this namespace.", namespace, maxAttempts);
143143
} else {
144144
log.info("The new LDD {} is indexed with date {}. It may take some time for it to be visible via mget.", namespace, info.lastDate);
145-
}
146-
145+
147146
// On AWS OpenSearch Serverless (AOSS), mget and search use different visibility paths.
148147
// Wait until a specific field document is also visible via mget with refresh=true,
149148
// so that getDataTypes() calls immediately following this load will succeed.
@@ -159,7 +158,8 @@ public void loadOnly(File lddFile, String lddFileName, String namespace, Instant
159158
}
160159
}
161160
}
162-
161+
log.info("Visibility of namespace " + namespace + "has been fully validated.");
162+
}
163163
} finally {
164164
// Delete temporary file
165165
tempEsDataFile.delete();

0 commit comments

Comments
 (0)