Possible resource leak?

You don’t have to modify Smack. You simply have to add something along these lines anywhere in Spark. This is the workaround to the current leak.

KeepAliveManager.addFailedPingListener(new PingFailedListener()

{

public void pingFailed()

{}

});

I say wait for the next Smack release next week or later. Only long runners are affected by this mostly, so probably not so critical.

By the way, to get older version of the jars, you can simply copy the url from the current download and paste it into a new tab and change the version.

http://www.igniterealtime.org/downloads/download-landing.jsp?file=smack/smack_3_ 3_0.zip

becomes

http://www.igniterealtime.org/downloads/download-landing.jsp?file=smack/smack_3_ 2_1.zip

Thanks for the assistance everyone. I’ll probably wait for the new smack. A post when it’s complete would be helpful. Thanks again!

Any better idea of the date for the release of the new version?

I am shooting for this weekend.

awesome, much appreciated!

Took a little longer than expected, but the new release is now out with the fix for the memory leak.

The powers that be should include this version as the dependency for Spark asap.

great, will test it and let everyone know!

SPARK-1552

Here’s the new installer (610 build with Smack 3.3.1) http://bamboo.igniterealtime.org/artifact/SPARK-INSTALL4J/JOB1/build-610/Install 4j/spark_2_6_3_12555.exe

great! thanks’ rcollier! and thanks wroot for taking care of the ticket and stuff.

i’m sure everything’s all good now, but i’ll post back the restults of my company’s most-recent interal build (probably rolling it out early next week).

Might be jumping the gun butI re-compiled it with the new smack but still running into the same problems. I’m going to retry building it and testing it again just incase I missed something. But just a heads up for anyone else testing it.

also make sure you do a full SVN checkout so that you get the latest Smack jar.

you can try applying this patch to your codebase: http://issues.igniterealtime.org/secure/attachment/12184/SPARK-33-Updated.patch

from SPARK-33

It updates the About Box and will display version info for Smack, among other things. The patch is more-or-less final, and is stable to run. This will help verify if you truely are using the latest Smack lib.

To apply the patch, I find using something like TortoiseSVN to make this easy. You just right-click the patch file, then click Apply Patch, choose the src directory of the Spark codebase, and it does the rest.

can we commit Spark-33 to the trunk?

Hello Walter. I’m OK with SPARK-33 as-is and use it in my company’s release with no problems. I think Tim Jentz was hinting at something about the build number from the CI being automatically updated somehow; I’m not familiar enough with Bamboo to know if that is possible… I did play around with ANT updating things, but it broke formatting horribly.

Anyways, I’m ok with it being committed now, we can always update it later as needed.

Hmm… I may be experiencing the same symptom as well. I just had my Spark hardlock on me and I had to force close it via Task manager. I restarted it, so we’ll keep an eye on it. I’m using the latest 3.3.1. Smack jar. Otherwise everything else is the same as it was before.

I loaded the version posted by wroot as is and it has been running successfully for a week now with no issues. Below 75mb of memory. I plan to do what Jason suggested and get a new copy of the SVN and update the Smack and then do my modifications. Will report back.

wroot, can you tell me how you built your installer? i did what Jason suggested but I’m still running into the same problem. your version works but my version with my modifications are still causing high java utilization. your assistance is appreciated.

It is built by Bamboo CI system on this site, not by me personally It just takes the source code and build it automatically (with Ant i think) when code changes are uploaded to SVN.