want bugfixes for 1.0.x version? about analytics-java HOT 6 CLOSED

@benhardy I can't imagine we wouldn't :) I'll defer to @f2prateek on the mechanics of how we'll bring it in and what branch to open it against, but just wanted to thank you for offering to contribute!

We'll be in touch shortly with next steps

from analytics-java.

Cheers, my pleasure.

from analytics-java.

@benhardy Can you share more details about the patch? Do you have a commit on Github?

from analytics-java.

Here's a diff against 1.0.7. All it does is make sure HttpResponses get closed. This was enough to ensure that connections in the pool can get reused efficiently. We saw after upgrading some services to Jetty 9.3. that the analytics library would just lock up because it was holding onto connections too long and timing out. This fixes that.

Commit: benhardy@b3656d1

Diff: analytics-1.0.7...benhardy:1.x

from analytics-java.

We're also operating with a fork that fixes #71. If there is going to be a 1.0.8 I'd be happy to create a PR so it could get in.

from analytics-java.

We'll make a 1.0.8 release to accomodate these fixes. I don't have an exact plan yet, but I'll post back with how to proceed.

I would still highly recommend using the v2 releases (even as a fork if you need Java 6 support). There are significant architectural changes and bug fixes for cases like this.

from analytics-java.