Skip to content

Conversation

@shubhamsrkdev
Copy link
Contributor

Description

Problem

Solution

  • Added toString() method in BytesRefBuilder.
  • Added UT for verifying the IllegalArgumentException exception.

@github-actions
Copy link
Contributor

This PR does not have an entry in lucene/CHANGES.txt. Consider adding one. If the PR doesn't need a changelog entry, then add the skip-changelog-check label to it and you will stop receiving this reminder on future updates to the PR.

Copy link
Contributor

@msfroh msfroh left a comment

Choose a reason for hiding this comment

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

Nice, thank you @shubhamsrkdev!

@msfroh
Copy link
Contributor

msfroh commented May 16, 2025

Oh, @shubhamsrkdev, could you please add an entry to lucene/CHANGES.txt? This is a fix that people will be able to experience.

@github-actions github-actions bot added this to the 11.0.0 milestone May 16, 2025
@shubhamsrkdev
Copy link
Contributor Author

Thanks @msfroh! I have added the change log entry.

@github-actions github-actions bot modified the milestones: 11.0.0, 10.3.0 May 19, 2025
@msfroh msfroh merged commit a5ab7e2 into apache:main May 19, 2025
7 checks passed
@shubhamsrkdev shubhamsrkdev deleted the addedTostring branch May 19, 2025 20:54
weizijun added a commit to weizijun/lucene that referenced this pull request May 27, 2025
* main: (32 commits)
  update os.makedirs with pathlib mkdir (apache#14710)
  Optimize AbstractKnnVectorQuery#createBitSet with intoBitset (apache#14674)
  Implement #docIDRunEnd() on PostingsEnum. (apache#14693)
  Speed up TermQuery (apache#14709)
  Refactor main top-n bulk scorers to evaluate hits in a more term-at-a-time fashion. (apache#14701)
  Fix WindowsFS test failure seen on Policeman Jenkins (apache#14706)
  Use a temporary repository location to download certain ecj versions ("drops") (apache#14703)
  Add assumption to ignore occasional test failures due to disconnected graphs (apache#14696)
  Return MatchNoDocsQuery when IndexOrDocValuesQuery::rewrite does not match (apache#14700)
  Minor access modifier adjustment to a couple of lucene90 backward compat types (apache#14695)
  Speed up exhaustive evaluation. (apache#14679)
  Specify and test that IOContext is immutable (apache#14686)
  deps(java): bump org.gradle.toolchains.foojay-resolver-convention (apache#14691)
  deps(java): bump org.eclipse.jgit:org.eclipse.jgit (apache#14692)
  Clean up how the test framework creates asserting scorables. (apache#14452)
  Make competitive iterators more robust. (apache#14532)
  Remove DISIDocIdStream. (apache#14550)
  Implement AssertingPostingsEnum#intoBitSet. (apache#14675)
  Fix patience knn queries to work with seeded knn queries (apache#14688)
  Added toString() method to BytesRefBuilder (apache#14676)
  ...
@jpountz
Copy link
Contributor

jpountz commented Aug 30, 2025

@msfroh Should this be backported to branch_10x?

@msfroh
Copy link
Contributor

msfroh commented Sep 2, 2025

@msfroh Should this be backported to branch_10x?

Oh -- it probably should. I'll take care of that.

@msfroh msfroh modified the milestones: 10.3.0, 10.4.0 Sep 2, 2025
@jpountz
Copy link
Contributor

jpountz commented Sep 4, 2025

@msfroh It looks like the CHANGES entry needs to move to 10.4 as well?

msfroh pushed a commit that referenced this pull request Sep 4, 2025
@msfroh
Copy link
Contributor

msfroh commented Sep 4, 2025

Okay -- backported this and moved the CHANGES entry. Thanks, @jpountz!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants