Skip to content
This repository has been archived by the owner on Oct 8, 2019. It is now read-only.

Feature/17 tracking processor rebalancing #14

Closed
wants to merge 10 commits into from

Conversation

saratry
Copy link
Contributor

@saratry saratry commented Aug 3, 2018

No description provided.

bool running = 4;
bool error = 5;
repeated EventTrackerInfo event_trackers_info = 6;
int32 active_threads = 3;
Copy link
Contributor

Choose a reason for hiding this comment

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

Backwards compatibility issue. When running this client version against previous server version, the server will not be able to parse the messages (as GRPC uses the sequence numbers).
Better to change the sequence nr for available threads to 7.

Copy link
Contributor

@MGathier MGathier left a comment

Choose a reason for hiding this comment

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

Approved if you change the sequence of the fields in the proto definition.

@saratry
Copy link
Contributor Author

saratry commented Sep 25, 2018

This feature will not be added to AxonClient. It is already implemented in axon-server-connector.

@saratry saratry closed this Sep 25, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants