Skip to content

Upgrade to Lucene 8.5.0-snapshot-c4475920b08#52950

Merged
dnhatn merged 4 commits intoelastic:masterfrom
dnhatn:upgrade-lucene
Feb 29, 2020
Merged

Upgrade to Lucene 8.5.0-snapshot-c4475920b08#52950
dnhatn merged 4 commits intoelastic:masterfrom
dnhatn:upgrade-lucene

Conversation

@dnhatn
Copy link
Copy Markdown
Member

@dnhatn dnhatn commented Feb 28, 2020

To give LUCENE-9228 more CI cycles.

@dnhatn dnhatn added >upgrade :Distributed/Engine Anything around managing Lucene and the Translog in an open shard. v8.0.0 v7.7.0 labels Feb 28, 2020
@dnhatn dnhatn requested a review from jpountz February 28, 2020 14:24
@elasticmachine
Copy link
Copy Markdown
Collaborator

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

@dnhatn dnhatn requested a review from jpountz February 28, 2020 14:39
@dnhatn
Copy link
Copy Markdown
Member Author

dnhatn commented Feb 28, 2020

@jpountz Can you please take another look? I've adjusted the QueryBuilder.

@dnhatn dnhatn requested a review from romseygeek February 28, 2020 14:41
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. We can remove our XIntervals implementation now as well, I think, but I'll do that in a followup.

@dnhatn
Copy link
Copy Markdown
Member Author

dnhatn commented Feb 29, 2020

@jpountz @romseygeek Thanks for reviews.

@dnhatn dnhatn merged commit 699b1dc into elastic:master Feb 29, 2020
@dnhatn dnhatn deleted the upgrade-lucene branch February 29, 2020 02:25
dnhatn added a commit to dnhatn/elasticsearch that referenced this pull request Feb 29, 2020
dnhatn added a commit that referenced this pull request Feb 29, 2020
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. >upgrade v7.7.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants