Java API questionable choice of Timestamp instead of Instant

[RobustCoder]

Java API questionable choice of Timestamp instead of Instant

[Svisstack]

I'm actually don't know the logic behind it as we did not maintain contact with the developer which wrote the JAVA SDK, however main question is: this create any issue? is this could be related to your previous issue with the timezone?

[carwilki]

java.sql.Timstamp is intended for use with jdbc stuff and is really not something that should be used...really ever at least out side of jdbc. since there are already other libraries required to use this then i would think that you can at least use Joda Time or if you only support jdk 1.8+ then you should be using the JSR-310 java.time implementation.

[RobustCoder]

carwilki: you are correct.

In the recent refactoring of coinapi's Java SDK, java.sql.Timstamp was totally replaced by java.time.Instant.

That change not only fixed the time zone bug reported in #36, it also simplified the code.

That refactoring addressed a bunch of other issues too. Check it out!