Skip to content

Conversation

@nblumhardt
Copy link
Member

Closes #318

The forwarder commands are behind a --pre flag, so we can move forward from here in two steps:

  1. Enumerate changes to non--pre functionality and fix before merging this PR
  2. Enumerate TODOs before we'll have a working forwarder implementation

liammclennan and others added 30 commits February 29, 2024 09:33
Merge in the existing Seq Forwarder codebase
Fix up forwarder build on Windows
Clean up the initial Forwarder code drop
@nblumhardt nblumhardt marked this pull request as ready for review July 18, 2025 07:10
@nblumhardt
Copy link
Member Author

Alrighty, I believe everything that's broken is now behind the --pre flag (and pretty much everything behind the --pre flag is broken - steady state ingestion appears to be fine, but there are unimplemented chunks in most forwarder-related things). All good fun for Monday! :-)

Copy link
Member

@KodrAus KodrAus left a comment

Choose a reason for hiding this comment

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

Thanks for getting this cleaned up! 👍

@nblumhardt nblumhardt merged commit d7bce51 into datalust:dev Jul 20, 2025
2 checks passed
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.

Support durable log forwarding in seqcli

3 participants