mbwaheed commented on a change in pull request #1430: URL: https://github.com/apache/lucene-solr/pull/1430#discussion_r425526455
########## File path: solr/core/src/test/org/apache/solr/store/shared/SharedCoreConcurrencyTest.java ########## @@ -595,7 +595,7 @@ private void configureTestSharedConcurrencyControllerForProcess( public void recordState(String collectionName, String shardName, String coreName, SharedCoreStage stage) { super.recordState(collectionName, shardName, coreName, stage); ConcurrentLinkedQueue<String> coreConcurrencyStages = coreConcurrencyStagesMap.computeIfAbsent(coreName, k -> new ConcurrentLinkedQueue<>()); - coreConcurrencyStages.add(Thread.currentThread().getId() + "." + stage.name()); + coreConcurrencyStages.add(Thread.currentThread().getName() + "." + stage.name()); Review comment: Sorry this is some what irrelevant. Looking through one of test run logs I realized in logging if thread has a name then that is logged e.g. puller threads. This change is only to help make debugging of this test easier. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org