SkygearIO / chat

Skygear Plugin - Chat SDK
Apache License 2.0
17 stars 18 forks source link

Pub/sub bug fix #207

Closed howawong closed 6 years ago

howawong commented 6 years ago

Connects #204

carmenlau commented 6 years ago

@howawong Would you mind help check the call of this function, if removing the limit is reasonable? The message event is sent in after_message_save async so I think it should be ok. But still would like to have your help to do a test and check if the message sending time reasonable when the room has more users (maybe around 3-400 people)?

carmenlau commented 6 years ago

After discussion, howa will help conduct a test in skygear cloud to check for the pubsub performance

howawong commented 6 years ago

Created a conversation with 500 participants in howachat. Got OOM issue, will test again.