Skip to content

[backport camel-4.14.x] CAMEL-24200: camel-aws2-timestream - createScheduledQuery sets the time column correctly#24949

Merged
davsclaus merged 1 commit into
apache:camel-4.14.xfrom
oscerd:fix/CAMEL-24200-4.14.x
Jul 20, 2026
Merged

[backport camel-4.14.x] CAMEL-24200: camel-aws2-timestream - createScheduledQuery sets the time column correctly#24949
davsclaus merged 1 commit into
apache:camel-4.14.xfrom
oscerd:fix/CAMEL-24200-4.14.x

Conversation

@oscerd

@oscerd oscerd commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Backport of #24909 to camel-4.14.x. Fixes CAMEL-24200 on the 4.14.x line.

The time-column header was written into the target TimestreamConfiguration's tableName instead of timeColumn, so the required time column was never set and a supplied table name was silently overwritten. Also removes a duplicated scheduledQueryExecutionRoleArn block.

Existing module tests extended in the original PR pass on this branch.


Claude Code on behalf of oscerd

…me column correctly (apache#24909)

Time-column header now sets timeColumn instead of overwriting tableName; removes a duplicated role-ARN block.

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
@oscerd
oscerd requested review from Croway and davsclaus July 20, 2026 13:12
@oscerd oscerd added the bug Something isn't working label Jul 20, 2026

@gnodet gnodet 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.

Clean backport of #24909 to camel-4.14.x — same time-column fix as #24948. Already reviewed and approved the original on main.

LGTM ✅

Claude Code on behalf of gnodet — AI-generated review

@davsclaus davsclaus added this to the 4.14.9 milestone Jul 20, 2026
@davsclaus
davsclaus merged commit 1e68e90 into apache:camel-4.14.x Jul 20, 2026
3 checks passed
@oscerd
oscerd deleted the fix/CAMEL-24200-4.14.x branch July 21, 2026 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants