Skip to content

ScanAll method at FullPrimaryKey, HashPrimaryKey#11

Open
extJo wants to merge 7 commits intoserverless-seoul:masterfrom
extJo:feature/scan-all
Open

ScanAll method at FullPrimaryKey, HashPrimaryKey#11
extJo wants to merge 7 commits intoserverless-seoul:masterfrom
extJo:feature/scan-all

Conversation

@extJo
Copy link

@extJo extJo commented Jul 8, 2020

BACKEND-STORY_ID One-sentence summary of changes

Is it a breaking change?: NO

Why did you make these changes?

When scan all records, for iteration, code is so dirty.
So, i think scanAll support at ORM layer.

What's changed in these changes?

I added scanAll method to FullPrimaryKey class.
I added scanAll method to HashPrimaryKey class.

What do you especially want to get reviewed?

Is parameter 'limit' of scanAll method needed?.

Is there any other comments that every teammate should know?

nope.

Submission Type

  • Bugfix
  • New Feature
  • Refactor

All Submissions

  • Have you added an explanation of what your changes?
  • Have you written new tests for your changes, as applicable?
  • Have you checked potential side effects that could make bad impacts to other services?

New Features

  • Have you configured CI/CD properly?
  • Have you configured optimal memory size and timeouts of Lambda Function?
  • Have you grant required permissions (e.g. IAM Policy, VPC Security Group)?
  • Have you made required resources (e.g. DynamoDB Table, RDS Cluster)?
  • Does it requires native addons dependency?

@extJo extJo requested a review from breath103 July 9, 2020 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants