Skip to content

SOLR-17448 SOLR-17497: IndexFetcher, catch exception instead of bubbling up uncaught#2800

Merged
iamsanjay merged 2 commits into
apache:mainfrom
iamsanjay:handlesAlreaedyClosed_IndexFetcher
Oct 31, 2024
Merged

SOLR-17448 SOLR-17497: IndexFetcher, catch exception instead of bubbling up uncaught#2800
iamsanjay merged 2 commits into
apache:mainfrom
iamsanjay:handlesAlreaedyClosed_IndexFetcher

Conversation

@iamsanjay
Copy link
Copy Markdown
Contributor

https://issues.apache.org/jira/browse/SOLR-17497

Checklist

Please review the following and check all that apply:

  • I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
  • I have created a Jira issue and added the issue ID to my pull request title.
  • I have given Solr maintainers access to contribute to my PR branch. (optional but recommended, not available for branches on forks living under an organisation)
  • I have developed this patch against the main branch.
  • I have run ./gradlew check.
  • I have added tests for my changes.
  • I have added documentation for the Reference Guide

@dsmiley dsmiley changed the title Add Exception Handling for Directory Closure During Sync in IndexFetcher SOLR-17448 SOLR-17497: IndexFetcher, catch exception instead of bubbling up uncaught Oct 29, 2024
Copy link
Copy Markdown
Contributor

@dsmiley dsmiley left a comment

Choose a reason for hiding this comment

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

In the merge commit, reference the

@iamsanjay iamsanjay merged commit cc30093 into apache:main Oct 31, 2024
iamsanjay added a commit that referenced this pull request Oct 31, 2024
gerlowskija pushed a commit that referenced this pull request Nov 4, 2024
HoustonPutman pushed a commit that referenced this pull request Apr 18, 2025
tboeghk pushed a commit to otto-de/solr that referenced this pull request Feb 18, 2026
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