nono303

No worries or rush on my side ;) Just ping me on this ticket if help or testing needed.

Hi @r_x, As I saw that you've already incorporated some of my commits into the trunk, I just attach a patch file for apacheCommonsHttp TileDownLoader, to avoid any dubious merge / rebase ;) In my pov., this implementation should be generalized because it resolves the TCP socket leak issue and significantly improves throughput (under the same conditions: trunk r2798 20MB/s vs. apacheCommonsHttp 54MB/s). I haven't found any memory leaks or issues after creating several atlases sequentially or simultaneously....

I’ll have a look onto sourceforge forking and merging but for now, we’ll continue diving on my github fork & once clean, I’ll rebase it on sourceforge branch with a PR (or give you a patch if it's a mess) Fyi, found an issue and fix it to use connManager on httpclient: https://github.com/nono303/mobac/commit/9907c20ce385e572ec80c523952ee38ee7906f2a (from 8.5MB/s to 51MB/s)

Many many thx @r_x for this work! I've just tested it: successful experiment! It burned my server cpu ^^ Same configuration as previous test: 24 parallel threads without local storage targeting local tile server on a 10 Gbps link result ~100000 tiles downloaded (4.4GB) in 3 minutes. On both Mobac and tile server side: netstat -an | grep TIME_WAIT | wc -l 0 I'll dive on TileDownLoader implementation trying to hardenize it for production readiness and fix some little new issue (ex. m.mapsources.custom.CustomMapSource:...

Thx @r_x for your quick feedback! adding conn.disconnect(); don't change behavior. Still have the blocking issue after ~16350 tiles downloaded I think that HttpURLConnection implementation didn't reuse TCP cnx at all Checking it with netstat comfort my thought > netstat -an | grep TIME_WAIT | wc -l 16353 playing with OS TCP parameters HKEY_LOCAL_MACHINE\SYSTEM\CurrentControlSet\Services\Tcpip\Parameters TcpTimedWaitDelay=10 TCPFinWait2Delay=10 MaxUserPort=65535 allow to have ~33000 tile downloaded...

Exception "Address already in use: getsockopt"

...I think my intends are not clear, I don't want to ignore or exclude theses file (I would just put them in another directory in this case) I want them to be loaded by mobac even if they had extra elements inside customMapSource root. An example, maybe be more clear: - this file works fine for mobac, geos, kmlsuperoverlay <?xml version="1.0" encoding="UTF-8"?> <customMapSource> <name><![CDATA[SwissTopo Rando [Overlay]]]></name> <minZoom>6</minZoom> <maxZoom>17</maxZoom> <tileType>png</tileType>...

...I think my intends are not clear, I don't want to ignore or exclude theses file (I would just put them in another directory in this case) I want them to be loaded by mobac even if they had extra elements inside customMapSource root. An example, maybe be more clear: - this file works fine for mobac, geos, kmlsuperoverlay <?xml version="1.0" encoding="UTF-8"?> <customMapSource> <name><![CDATA[SwissTopo Rando [Overlay]]]></name> <minZoom>6</minZoom> <maxZoom>17</maxZoom> <tileType>png</tileType>...