Skip to content

add stream encoder#55

Merged
HDT3213 merged 4 commits intoHDT3213:masterfrom
MIcQo:feat/stream_encoder
Sep 14, 2025
Merged

add stream encoder#55
HDT3213 merged 4 commits intoHDT3213:masterfrom
MIcQo:feat/stream_encoder

Conversation

@MIcQo
Copy link
Contributor

@MIcQo MIcQo commented Aug 21, 2025

  • Add option to write stream object into RDB file with tests
  • Add to encoder example

@MIcQo MIcQo force-pushed the feat/stream_encoder branch from cc3ff70 to 3a7fff2 Compare August 21, 2025 15:03
@MIcQo MIcQo mentioned this pull request Aug 21, 2025
@MIcQo
Copy link
Contributor Author

MIcQo commented Aug 21, 2025

@HDT3213 can you look at it?

@MIcQo MIcQo changed the title add stream encoder WIP: add stream encoder Aug 21, 2025
@MIcQo MIcQo changed the title WIP: add stream encoder add stream encoder Aug 21, 2025
Entries: []*model.StreamEntry{}, // Empty entries
Groups: []*model.StreamGroup{}, // Empty groups
}
err = encoder.WriteStreamObject("mystream", stream)
Copy link
Owner

Choose a reason for hiding this comment

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

undefined: encoder.WriteStreamObject

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@HDT3213 yea, because it uses repository as import package, but repository not implement WriteStreamObject yet. What is your recon to resolve this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@HDT3213 can you look at it again?

@MIcQo MIcQo requested a review from HDT3213 September 8, 2025 20:05
@HDT3213 HDT3213 merged commit b5e024c into HDT3213:master Sep 14, 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.

2 participants