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

Followup of #817 #824

Merged
merged 2 commits into from
Jun 12, 2020
Merged

Followup of #817 #824

merged 2 commits into from
Jun 12, 2020

Conversation

apeksharma
Copy link
Contributor

@apeksharma apeksharma commented Jun 11, 2020

Detailed description:

  • Change Long to EntityId in Transaction and Entities. Added EntityIdConverter for the same.
  • Use only id field in EntityId for EqualsAndHashCode since that is sufficient
  • Add InvalidEntityException. Thrown for invalid entities when encoding/decoding
  • Handle the case for invalid entityId (contract/topic/file id) in a failed transaction. Added test.

Signed-off-by: Apekshit Sharma [email protected]

Which issue(s) this PR fixes:
Fixes #570
Part of #820

Special notes for your reviewer:

Checklist

  • Documentation added
  • Tests updated

@apeksharma apeksharma mentioned this pull request Jun 11, 2020
2 tasks
@apeksharma apeksharma self-assigned this Jun 11, 2020
@apeksharma apeksharma added enhancement Type: New feature P2 parser Area: File parsing labels Jun 11, 2020
@apeksharma apeksharma added this to the Mirror 0.14.0 milestone Jun 11, 2020
@apeksharma apeksharma marked this pull request as ready for review June 11, 2020 22:29
Signed-off-by: Apekshit Sharma <[email protected]>
Copy link
Contributor

@steven-sheehy steven-sheehy left a comment

Choose a reason for hiding this comment

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

Just some minor suggestions

Nana-EC
Nana-EC previously approved these changes Jun 12, 2020
Signed-off-by: Apekshit Sharma <[email protected]>
Copy link
Contributor

@Nana-EC Nana-EC left a comment

Choose a reason for hiding this comment

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

LGTM

@apeksharma apeksharma merged commit 1cb9055 into master Jun 12, 2020
@apeksharma apeksharma deleted the denorm_followup branch June 12, 2020 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Type: New feature P2 parser Area: File parsing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Denormalize t_transactions table
3 participants