Skip to content

Conversation

@liuguoqingfz
Copy link
Contributor

No description provided.

gr-data.csv Outdated
https://github.com/opensearch-project/OpenSearch,e0d622ddc73bdab552d94427dfeb3e101b026ce2,OpenSearch/server/src/test/java/org/opensearch/index/fielddata,org.opensearch.index.fielddata.IndexFieldDataServiceTests.testIndexFieldDataCacheIsClearedAfterIndexRemoval,NOD,Opened,https://github.com/opensearch-project/OpenSearch/pull/19759,
https://github.com/opensearch-project/OpenSearch,3506973761972df391cabef40b9858e6c05239f0,OpenSearch/modules/mapper-extras/src/test/java/org/opensearch/index/mapper,org.opensearch.index.mapper.ScaledFloatFieldMapperTests.testScaledFloatWithStarTree,NOD,Opened,https://github.com/opensearch-project/OpenSearch/pull/20254,
https://github.com/opensearch-project/OpenSearch,bd98dfb3e5fee6bd8c88667841088149dff5da33,OpenSearch/server/src/internalClusterTest/java/org/opensearch/index/shard,org.opensearch.index.shard.IndexShardIT.testIndexCanChangeCustomDataPath,NOD,,,
https://github.com/opensearch-project/OpenSearch,c3533a7c2a3c17d2dfadb270b60d1c8006b687dd,OpenSearch/server/src/internalClusterTest/java/org/opensearch/indices/replication,org.opensearch.indices.replication.MergedSegmentWarmerIT.testPrimaryStopped_ReplicaPromoted,NOD,Deleted,,https://github.com/opensearch-project/OpenSearch/issues/19042
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What commit deleted this test? Your not has some issue that talks about several tests.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do you mean by "Your not has some issue that talks about several tests."? Do you need me to find out which SHA was this test deleted?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, find the commit SHA that deleted the test if you claim the status Deleted. Sorry for the typo: your note has a link to some issue that talks about several tests. The note should be a link to the commit that deleted this one test.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The method was actually in the parent class, I couldn't find the method in the child class so I thought it was deleted. I've opened a PR for the fix in the parent class, so I changed Deleted to Opened, and there was another test that has the same situation as this one but I don't think I have enough time to fix that so I just change that Deleted to Blank

Change deleted to opened
@darko-marinov darko-marinov merged commit 69dc007 into TestingResearchIllinois:main Dec 17, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants