Skip to content

Don't use endianness reverse util for writing metadata state file#78309

Merged
iverase merged 1 commit intoelastic:masterfrom
iverase:MetadataStateFormat
Sep 27, 2021
Merged

Don't use endianness reverse util for writing metadata state file#78309
iverase merged 1 commit intoelastic:masterfrom
iverase:MetadataStateFormat

Conversation

@iverase
Copy link
Copy Markdown
Contributor

@iverase iverase commented Sep 27, 2021

When Elasticsearch was upgraded to Lucene 9.0, the metadata state file mechanism for files was not changed and we are just reversing the endianness of the files for reading and writing. On the long term we want to write the file in the endianness of the data input so this PR does that. New files are writing in LE and a new version of the files is added. We handle the different endianness when reading files.

follow up of #73324

@iverase iverase added >non-issue :Search/Search Search-related issues that do not fall into other categories :Distributed/Engine Anything around managing Lucene and the Translog in an open shard. v8.0.0 labels Sep 27, 2021
@iverase iverase requested a review from romseygeek September 27, 2021 10:16
@elasticmachine elasticmachine added Team:Search Meta label for search team Team:Distributed Meta label for distributed team. labels Sep 27, 2021
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-search (Team:Search)

@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed (Team:Distributed)

@iverase iverase mentioned this pull request Sep 27, 2021
16 tasks
Copy link
Copy Markdown
Contributor

@romseygeek romseygeek left a comment

Choose a reason for hiding this comment

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

LGTM

@iverase iverase merged commit 28c74a0 into elastic:master Sep 27, 2021
@iverase iverase deleted the MetadataStateFormat branch September 27, 2021 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Engine Anything around managing Lucene and the Translog in an open shard. >non-issue :Search/Search Search-related issues that do not fall into other categories Team:Distributed Meta label for distributed team. Team:Search Meta label for search team v8.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants