Skip to content

Fixed the ingestion type parameter for DynamoDB data jobs#598

Merged
juanpardo merged 1 commit into
masterfrom
jp/fixed-ingestion-type-param
Jul 21, 2026
Merged

Fixed the ingestion type parameter for DynamoDB data jobs#598
juanpardo merged 1 commit into
masterfrom
jp/fixed-ingestion-type-param

Conversation

@juanpardo

Copy link
Copy Markdown
Contributor

Summary of the changes / Why this is an improvement

@coderabbitai

coderabbitai Bot commented Apr 1, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 12cc2510-cac5-43c0-a51c-e73179832635

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch jp/fixed-ingestion-type-param

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@juanpardo
juanpardo requested a review from plaharanne April 1, 2026 07:35

@plaharanne plaharanne left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you add it to the changelog too?

@juanpardo

Copy link
Copy Markdown
Contributor Author

Can you add it to the changelog too?

The changelog already has a Added support for DynamoDB data jobs entry, and hasn't been released yet, that's why I thought it would feel redundant to have a feature and a fix for the feature in the same release changelog entry. What do you think @plaharanne ?

@plaharanne

Copy link
Copy Markdown
Contributor

Hey @juanpardo I think you can merge this pull request. It is opened for a while

@juanpardo
juanpardo force-pushed the jp/fixed-ingestion-type-param branch from f81ae82 to 32ae10e Compare July 21, 2026 13:32
@juanpardo
juanpardo merged commit 925cc3d into master Jul 21, 2026
12 checks passed
@juanpardo
juanpardo deleted the jp/fixed-ingestion-type-param branch July 21, 2026 13:34
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