search

jimmoores / quandl4j

Java wrapper for Quandl REST API
quandl4j.org
Apache License 2.0
78 stars 22 forks source link

RESTEASY004655 error #25

Closed oasuncion closed 7 years ago

oasuncion commented 7 years ago

Hello,

First, thank you for this very useful library.

I tested Quandl4j within a java EE application and got the error attached when several clients were requesting data:

log-RESTEASY004655.txt

The error can be fixed closing the response object in DefaultRESTDataProvider#getTabularResponse once it is used.

My environment: JDK 1.8.0 update 102 Application server : Widlfly 10.1.0.Final OS : Windows 7 Quandl4j : 1.4.0

Thanks and regards, Olivier

jimmoores commented 7 years ago

Hi Olivier,

Thank you so much for taking the time to let me know about your problem, and doubly so for providing a solution! I'll be integrating your suggested fix tomorrow. If you don't have any objections, I'll also add you to the list of contributors - I believe in acknowledging all help in the project. I'll then follow up with a new release in the coming days.

Best Regards,

Jim

oasuncion commented 7 years ago

Hi Jim,

Thanks for the quick reply and for the release. I'll be glad to be added to the list of contributors, thanks!

Best regards, Olivier

oasuncion commented 7 years ago

Hi Jim, You must be extremely busy but would it be possible to get a v1.4.1 with this fix ? Best regards, Olivier

jimmoores commented 7 years ago

I shall get it out today. Apologies for the delay. I'll let you know when it's done.

On Thu, 20 Apr 2017 at 07:37, Olivier-92 notifications@github.com wrote:

Hi Jim, You must be extremely busy but would it be possible to get a v1.4.1 with this fix ? Best regards, Olivier

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/jimmoores/quandl4j/issues/25#issuecomment-295598573, or mute the thread https://github.com/notifications/unsubscribe-auth/AADG-45qiZFK1zVVCdrQAgv0F3KF59v-ks5rxv0xgaJpZM4MTBWe .

jimmoores commented 7 years ago

I've updated the docs, poms, etc and deployed the release to maven central. It does take a little while to show up in the search (I'll double check a bit later but it's not showing up yet).

oasuncion commented 7 years ago

Thanks you very much, I retrieved the v1.4.1, integrated it and can confirm that I don't encounter this issue anymore. I close this ticket. BR, Olivier