Skip to content

Changing big_map_contents PK so we avoid the issue with the too large key#1065

Merged
SiddharthV1 merged 1 commit intomasterfrom
bugfix/changing-primary-key-in-big_map_contents
Jan 21, 2022
Merged

Changing big_map_contents PK so we avoid the issue with the too large key#1065
SiddharthV1 merged 1 commit intomasterfrom
bugfix/changing-primary-key-in-big_map_contents

Conversation

@piotrkosecki
Copy link
Copy Markdown
Contributor

@piotrkosecki piotrkosecki commented Jan 21, 2022

Changed PK to consist of key_hash instead of key, which will be smaller.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Jan 21, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 32 Code Smells

No Coverage information No Coverage information
4.2% 4.2% Duplication

@SiddharthV1 SiddharthV1 merged commit be6bffd into master Jan 21, 2022
@vishakh vishakh added this to the 2022_r1 milestone Mar 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vishakh vishakh Awaiting requested review from vishakh

@bhaskarkishore bhaskarkishore Awaiting requested review from bhaskarkishore

1 more reviewer

@SiddharthV1 SiddharthV1 SiddharthV1 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

2022_r1

Development

Successfully merging this pull request may close these issues.

3 participants

@piotrkosecki @SiddharthV1 @vishakh