Skip to content
This repository has been archived by the owner on Jun 20, 2024. It is now read-only.

Streamクラスの実装が当初考えていた機能と比べて不十分 #124

Closed
marihachi opened this issue Oct 5, 2018 · 1 comment

Comments

@marihachi
Copy link
Member

marihachi commented Oct 5, 2018

よくよく考えてみると、
「Redisの複数のチャンネルに流れてくる内容をマージしたRedisのチャンネルを作り出す機能」を持つはずが、その作り出したチャンネルがNode標準のEventEmitterによる実装になっていて、あまりよろしくない。

@marihachi
Copy link
Member Author

#125 をする上でこの変更は都合が悪いことに気づいた。
ノードごとにStreamを生成するのは仕方ないとして、
Redisに、それぞれのノードがマージしたチャンネルを作るとなると、重複してメッセージが投げられることになる(はず)
それだとまずい。

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant