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
Use bulk update API for shard allocation #1139
Conversation
@@ -9,7 +9,7 @@ val Nightly = sys.env.get("TRAVIS_EVENT_TYPE").contains("cron") | |||
val Scala211 = "2.11.12" | |||
val Scala212 = "2.12.10" | |||
val Scala213 = "2.13.1" | |||
val akkaVersion26 = "2.6.3" | |||
val akkaVersion26 = "2.6.6" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this upgrade is required? If not we'd keep 2.6.3.
Aboid multiple ddata round trips
The new API is not in 2.6.3 but I could check what the min upgrade could be
if that helps? But if we’re upgrading maybe we should just go to 2.6.6
…On Tue, 7 Jul 2020 at 11:26, Enno ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In build.sbt
<#1139 (comment)>:
> @@ -9,7 +9,7 @@ val Nightly = sys.env.get("TRAVIS_EVENT_TYPE").contains("cron")
val Scala211 = "2.11.12"
val Scala212 = "2.12.10"
val Scala213 = "2.13.1"
-val akkaVersion26 = "2.6.3"
+val akkaVersion26 = "2.6.6"
Is this upgrade is required? If not we'd keep 2.6.3.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#1139 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAOHYG7NPCCKQ52Y2EZKQGTR2LZ5DANCNFSM4OSQ2DGQ>
.
|
Yes, when we update we go all the way. |
f3dbafb
to
fddc156
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Unrelated build failure: #1070 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
On top of #1138