Skip to content

Conversation

@janhoy
Copy link
Contributor

@janhoy janhoy commented Oct 14, 2025

@github-actions github-actions bot added documentation Improvements or additions to documentation module:extraction labels Oct 14, 2025
@janhoy janhoy requested a review from Copilot October 14, 2025 22:35
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR deprecates the XLSXResponseWriter class in Solr v9.10 as indicated by JIRA ticket SOLR-17956. The deprecation includes proper documentation updates and follows established deprecation practices.

Key changes:

  • Added @Deprecated annotation to XLSXResponseWriter class with removal flag
  • Updated documentation to inform users about the deprecation
  • Added change log entry documenting the deprecation

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
XLSXResponseWriter.java Added deprecation annotation and updated javadoc to indicate future removal
response-writers.adoc Added important notice about XLSXResponseWriter deprecation
major-changes-in-solr-9.adoc Added deprecations section documenting XLSXResponseWriter deprecation
CHANGES.txt Added changelog entry for the deprecation

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@janhoy janhoy requested a review from dsmiley October 14, 2025 22:36
@janhoy janhoy force-pushed the SOLR-17956-deprecate-XLSXResponseWriter branch from 9989938 to 780001a Compare October 15, 2025 12:59
@janhoy janhoy merged commit dead9b6 into main Oct 15, 2025
3 checks passed
@janhoy janhoy deleted the SOLR-17956-deprecate-XLSXResponseWriter branch October 15, 2025 20:30
janhoy added a commit that referenced this pull request Oct 15, 2025
janhoy added a commit that referenced this pull request Oct 15, 2025
janhoy added a commit to janhoy/solr that referenced this pull request Oct 15, 2025
Backported from main branch dead9b6

Signed-off-by: Jan Høydahl <jan.git@cominvent.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Dependency upgrades documentation Improvements or additions to documentation module:extraction tests tool:build

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants