commit b4c78fbb5a4996ee7b1483c846bf58d685cb1213
author Antonio Barone <syntonyze@gmail.com> Tue Jun 29 16:19:21 2021 +0200
committer Antonio Barone <syntonyze@gmail.com> Wed Jun 30 12:03:58 2021 +0200
tree 79ecf5ecdb60a6e880386cf9667fc23b09170392
parent 77bb78c662f8c25f55b1294ef58de8a7f537f68f

Stop using Gerrit instanceId as partition key

Using the Gerrit instanceId as partition key might lead to
IllegalArgument exceptions, due to the fact that it can be null (since
it is an optional configuration).

Moreover, using _only_ the instanceId would lead to a suboptimal
sharding of the messages across different kinesis shards, in case where
the majority of messages would come from one active gerrit instance.

Set the partition key to the event type, which is always present and
provides a better distribution than instanceId.

Bug: Issue 14697
Change-Id: I467e79d8c98958cc68911a54a684ae35afe26547