Skip to content

feat: Remove write_disposition from opralogweb configuration#186

Merged
martyngigg merged 1 commit into
mainfrom
elt-opralogweb-incremental
Jan 7, 2026
Merged

feat: Remove write_disposition from opralogweb configuration#186
martyngigg merged 1 commit into
mainfrom
elt-opralogweb-incremental

Conversation

@martyngigg

@martyngigg martyngigg commented Jan 7, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Member

Summary

Use default write_disposition now incremental loading has been correctly implemented.

Refs #138

Summary by CodeRabbit

  • Configuration Updates
    • Simplified data warehouse write operation configuration to enhance consistency in data handling processes.

✏️ Tip: You can customize this high-level summary in your review settings.

@martyngigg
Remove write_disposition from opralogweb configuration
d15dc95 
Use default write_disposition now incremental loading has
been correctly implemented
@coderabbitai

coderabbitai Bot commented Jan 7, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The write_disposition: "replace" field was removed from the opralogweb warehouse entry within the elt_warehouses configuration in the Ansible variables file. This eliminates the explicit write disposition setting previously applied to that source.

Changes

Cohort / File(s) Summary
ELT Configuration
infra/ansible-docker/playbooks/elt/group_vars/elt.yml		 Removed write_disposition: "replace" field from opralogweb entry under elt_warehouses

Poem

🐰 A field takes flight, the config grows lean,
Write disposition fades from the scene,
Opralogweb now lighter, more free,
Simplicity hops with glee! 🐇

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Cache: Disabled due to data retention organization setting

Knowledge base: Disabled due to data retention organization setting

📥 Commits

Reviewing files that changed from the base of the PR and between a38d34d and d15dc95.

📒 Files selected for processing (1)
  • infra/ansible-docker/playbooks/elt/group_vars/elt.yml

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 and usage tips.

@martyngigg
martyngigg enabled auto-merge (squash) January 7, 2026 15:29
@martyngigg
martyngigg merged commit d0ab333 into main Jan 7, 2026
1 of 2 checks passed
@martyngigg
martyngigg deleted the elt-opralogweb-incremental branch January 7, 2026 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@martyngigg