Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BREAKING: Changed GetFilePointer() methods into Position properties #483

Merged
merged 2 commits into from
May 3, 2021

Conversation

NightOwl888
Copy link
Contributor

This changes GetFilePointer() methods to properties named Position, which matches the convention used by System.IO.FileStream.Position. Affected types:

  • Lucene.Net.Store.IndexInput (and subclasses)
  • Lucene.Net.Store.IndexOutput (and subclasses)

Extension methods were also added in the Lucene.Net.Store namespace for backward compatibility. These extension methods are hidden from Intellisense and marked obsolete. They will be removed in the first 4.8.0 release candidate.

NOTE: These are meant to be a best effort to reduce breaking changes, but of course the Lucene.Net.Store namespace may not be imported in your code. It is recommended to utilize the property rather than importing the Lucene.Net.Store namespace.

This also changes a call in ReadersAndUpdates from the obsolete BinaryDocValuesField.fType to BinaryDocValuesField.TYPE (which was missed in #482).

…ld.TYPE instead of BinaryDocValuesField.fType.
…Position to match FileStream. Types affected: IndexInput, IndexOutput. Added obsolete extension methods for each type in Lucene.Net.Store, but do note this only works if the Lucene.Net.Store namespace is imported at the top of the file.
@NightOwl888 NightOwl888 added this to the 4.8.0-beta00015 milestone May 3, 2021
@NightOwl888 NightOwl888 merged commit fb04740 into apache:master May 3, 2021
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.

1 participant