Skip to content

Conversation

@sandykumar93
Copy link
Contributor

@sandykumar93 sandykumar93 commented Sep 19, 2024

Issue #2482

Description of changes:
The newly added pattern shows how to use Lambda response streaming to incrementally retrieve and stream DynamoDB query / scan results using the write() method. Instead of waiting for the entire query / scan operation to complete, the Lambda function streams data in batches by setting a limit on the number of items per query and sending each batch as soon as it's retrieved. This improves the time-to-first-byte (TTFB) by streaming results to the client as they become available.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Copy link
Contributor

@ellisms ellisms left a comment

Choose a reason for hiding this comment

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

Thanks for your submission @sandykumar93 . Requesting a few minor edits.

@sandykumar93
Copy link
Contributor Author

sandykumar93 commented Sep 24, 2024

@ellisms thank you for taking the time to review and provide suggestions. I have applied all the suggestions in the latest commit. I have resolved all the comments except the one for data population.

Please have a look when possible and let me know if any other changes are required.

Copy link
Contributor

@ellisms ellisms left a comment

Choose a reason for hiding this comment

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

Just one last edit.

ellisms and others added 2 commits September 24, 2024 08:27
Co-authored-by: ellisms <114107920+ellisms@users.noreply.github.com>
@ellisms
Copy link
Contributor

ellisms commented Sep 24, 2024

Thanks, adding to the publishing queue.

@julianwood julianwood merged commit f186dee into aws-samples:main Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants