Skip to content

Commit d6585a6

Browse files
csviriCopilot
andcommitted
Apply suggestions from code review
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
1 parent aa978c2 commit d6585a6

1 file changed

Lines changed: 2 additions & 6 deletions

File tree

  • operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/event/source/informer/pool

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/event/source/informer/pool/AbstractInformerPool.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -133,11 +133,7 @@ public <R extends HasMetadata> void start(
133133
final var name = thread.getName();
134134
try {
135135
thread.setName(
136-
"InformerInfo[informerInfo"
137-
+ informer.getApiTypeClass().getSimpleName()
138-
+ "]"
139-
+ " "
140-
+ thread.getId());
136+
"InformerInfo[" + informer.getApiTypeClass().getSimpleName() + "] " + thread.getId());
141137
final var resourceName = informer.getApiTypeClass().getSimpleName();
142138
// idempotent: if the informer was already started (e.g. by the pool when it was
143139
// created/reused), this just returns the existing start future without restarting it
@@ -179,7 +175,7 @@ public <R extends HasMetadata> void start(
179175

180176
private String versionedFullResourceName(SharedIndexInformer<? extends HasMetadata> informer) {
181177
final var apiTypeClass = informer.getApiTypeClass();
182-
if (apiTypeClass.isAssignableFrom(GenericKubernetesResource.class)) {
178+
if (GenericKubernetesResource.class.isAssignableFrom(apiTypeClass)) {
183179
return GenericKubernetesResource.class.getSimpleName();
184180
}
185181
return ReconcilerUtilsInternal.getResourceTypeNameWithVersion(apiTypeClass);

0 commit comments

Comments
 (0)