Skip to content

Conversation

bennetthardwick
Copy link
Contributor

Changelog

None

Docs

None

Description

Adds another example data loader for loading from a CSV. It uses an index to keep track of where to start reading from. It logs each column as a different topic and uses a "timestamp_nanos" column for the log time.

@bennetthardwick bennetthardwick force-pushed the bennett/add-csv-data-loader branch from 234f8d7 to f525b09 Compare August 13, 2025 06:21
@bennetthardwick bennetthardwick marked this pull request as ready for review August 14, 2025 04:55
Copy link

@nidanin nidanin left a comment

Choose a reason for hiding this comment

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

Seems ok to me. Left a few minor comments.

@bennetthardwick bennetthardwick merged commit c34b191 into main Aug 15, 2025
4 checks passed
@bennetthardwick bennetthardwick deleted the bennett/add-csv-data-loader branch August 15, 2025 05:09
bennetthardwick added a commit that referenced this pull request Aug 20, 2025
### Changelog
<!-- Write a one-sentence summary of the user-impacting change (API,
UI/UX, performance, etc) that could appear in a changelog. Write "None"
if there is no user-facing change -->

None

### Docs

<!-- Link to a Docs PR, tracking ticket in Linear, OR write "None" if no
documentation changes are needed. -->

None

### Description

These changes were supposed to be in #179 but weren't pushed before
merging.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants