Re: Type annonations in clojure.instant/print-date result in speedup of 3

2013-10-10 Thread Alexander Kiel
Thanks Andy. I added a comment to http://dev.clojure.org/jira/browse/CLJ-1277 Am Donnerstag, 10. Oktober 2013 07:12:47 UTC+2 schrieb Andy Fingerhut: > > Thanks for checking, Alexander. I have created ticket CLJ-1277 with a > patch that improves the performance of the case you found, as well as

Re: Type annonations in clojure.instant/print-date result in speedup of 3

2013-10-09 Thread Andy Fingerhut
Thanks for checking, Alexander. I have created ticket CLJ-1277 with a patch that improves the performance of the case you found, as well as for java.sql.Timestamp printing that also had reflection in it nearly identical to what you found. http://dev.clojure.org/jira/browse/CLJ-1277 Feel free

Re: Type annonations in clojure.instant/print-date result in speedup of 3

2013-10-09 Thread Alexander Kiel
Hi Andy, http://dev.clojure.org/jira/browse/CLJ-1080 has not the same type hints as I propose. You can see my patch here: https://github.com/alexanderkiel/clojure/compare/instant-type-hints I have an additional type hint at thread-local-utc-date-format and I use java.text.DateFormat instea

Re: Type annonations in clojure.instant/print-date result in speedup of 3

2013-10-09 Thread Andy Fingerhut
Could you take a look at the patch on ticket CLJ-1080 and see if it contains all of the improvements you mention? Let me know if it doesn't, and we can look at combining your improvements into that patch. http://dev.clojure.org/jira/browse/CLJ-1080 Andy On Wed, Oct 9, 2013 at 8:57 AM, Alex

Type annonations in clojure.instant/print-date result in speedup of 3

2013-10-09 Thread Alexander Kiel
Hi, I have a web service returning edn representations containing many java.util.Date values. Using a profiler, one Hot Spot is inside the clojure.instant/print-date function. The relevant reflections warnings are: Reflection warning, instant.clj