Skip to content

Conversation

@martyngigg
Copy link
Contributor

@martyngigg martyngigg commented Jan 7, 2026

Summary

Script doesn't work reliably through dlt. Remove and rewrite in the future.

Summary by CodeRabbit

  • Revert
    • Removed the InfluxDB extract and load accelerator module, including configuration, pipeline code, dependencies, and documentation.

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

Script doesn't work reliably through dlt.
Remove and rewrite in the future
@coderabbitai
Copy link

coderabbitai bot commented Jan 7, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Removes the entire InfluxDB extract/load accelerator, including configuration, pipeline module, documentation, and dependencies. The accelerator previously facilitated InfluxDB data extraction with filesystem destination support and multi-schema orchestration.

Changes

Cohort / File(s) Summary
InfluxDB Accelerator Removal
warehouses/accelerator/extract_load/influxdb/.dlt/config.toml, warehouses/accelerator/extract_load/influxdb/elt_influxdb_machinestate.py, warehouses/accelerator/extract_load/influxdb/readme.md, warehouses/accelerator/extract_load/influxdb/requirements.in, warehouses/accelerator/extract_load/influxdb/requirements.txt
Complete deletion of InfluxDB accelerator including runtime/destination configuration, ELT pipeline orchestration (InfluxQuery helper, machinestate extraction, multi-process schema handling), documentation, and all pinned and source dependencies.

Poem

🐰 A carrot path once led through InfluxDB's gate,
With channels and schemas, now sealed by fate,
Config, code, and dependencies fall away,
The accelerator rests, its work done this day,
Into the rabbit hole of history's array. 🕳️


📜 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 6dc1992 and cdd0903.

📒 Files selected for processing (5)
  • warehouses/accelerator/extract_load/influxdb/.dlt/config.toml
  • warehouses/accelerator/extract_load/influxdb/elt_influxdb_machinestate.py
  • warehouses/accelerator/extract_load/influxdb/readme.md
  • warehouses/accelerator/extract_load/influxdb/requirements.in
  • warehouses/accelerator/extract_load/influxdb/requirements.txt

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 16:49
@martyngigg martyngigg merged commit acfe73c into main Jan 7, 2026
1 of 2 checks passed
@martyngigg martyngigg deleted the rm-influx-extract-and-load branch January 7, 2026 16:49
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