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

-> v1.7.0-RC4 #1592

Merged
merged 1 commit into from
Apr 22, 2021
Merged

-> v1.7.0-RC4 #1592

merged 1 commit into from
Apr 22, 2021

Conversation

mhowlett
Copy link
Contributor

-> 1.7.0-RC4. Also updated `System.Net.Http to v4.3.4 (CVE-2018-8292). Also improved rebalance console output in the consumer example, which I have been using to observe partition shuffling.

@mhowlett mhowlett requested a review from edenhill April 22, 2021 18:29
@@ -65,7 +65,12 @@ public static void Run_Consume(string brokerList, List<string> topics, Cancellat
{
// Since a cooperative assignor (CooperativeSticky) has been configured, the
// partition assignment is incremental (adds partitions to any existing assignment).
Console.WriteLine($"Incremental partition assignment: [{string.Join(", ", partitions)}]");
Console.WriteLine(
"Partitions incrementally assigned: [" +
Copy link
Contributor

Choose a reason for hiding this comment

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

Next time please use an individual commit for this since it is not related to the RC bump

@@ -27,7 +27,7 @@

<ItemGroup>
<PackageReference Include="Newtonsoft.Json" Version="9.0.1" />
<PackageReference Include="System.Net.Http" Version="4.3.0" />
<PackageReference Include="System.Net.Http" Version="4.3.4" />
Copy link
Contributor

Choose a reason for hiding this comment

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

And for this one in particular.

@mhowlett mhowlett merged commit 85facb6 into confluentinc:1.7.x Apr 22, 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.

2 participants