[#SMACK-129] MultiUserChat will Store Messages in its PacketCollector irregardless of whether or not they are being read

Smack

MultiUserChat will Store Messages in its PacketCollector irregardless of whether or not they are being read

Created: 03/29/06 09:05 PM Updated: 05/17/08 08:40 AM

Component/s:

Extensions

Affects Version/s:

2.0.0, 2.1.0, 2.2.0

Fix Version/s:

None

Time Tracking:

Not Specified

Issue Links:

Related to

This issue is related to:

~~SMACK-6~~ Don't force use of collectors in Chat, GroupChat, etc

Support Plan Customer Issue:	No
Acceptance Test - Add?:	No

Description

« Hide

MultiUserChat will store messages in a room, if many messages are recieved it will cause Smack to run out of memory.

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

Ascending order - Click to sort in descending order

[ Permlink | « Hide ]

Matt Tucker - 02/04/07 12:54 PM

We should likely make the fix like the one that Alex implemented for Chat – force passing in a listener when creating the room. Creating the collector should be optional.

[ Permlink | « Hide ]

Joshua Davis - 05/17/08 08:40 AM

I see that MutiUserChat creates a ConnectionDetachedPacketListener which keeps a list of 65536 packets (MAX_PACKETS). Maybe that should be configurable rather than being a private static final field?