[CL Incentives] Key incentive records by owner address to prevent DoS/trolling

[AlpinYukseloglu]

Background

We currently key incentive records by poolId, denom, and minUptime. While this is technically sufficient and robust, it allows for any user to dust each uptime accumulator with a denom and essentially prevent that denom from being used for incentives.

This became much bigger of an issue when we constrained the supported uptimes, since it becomes trivially easy to do this.

Suggested Design

• I suggest we add incentiveOwner to the end of the incentive record key. This prevents anyone from being blocked from creating incentives due to existing incentives on the pool. This will require updating incentive record getters and setters.
• To ensure incentive retrieval remains simple, I suggest we move minUptime in front of denom in the key and then implement a GetAllIncentiveRecordsForUptime function that gathers all the incentives for a given poolId | minUptime from state

Acceptance Criteria

• All new and existing tests pass