Dave Cramer comments

Results 939 comments of


                                            Dave Cramer

fix: #1431: memory leak in UTF8 string decoding

> Seeking inputs if the below will be a good fix 1. Introduce a connection level param MAX_DECODER_ARRAY_LEN with a default value of say 1 MB. Add getter/setter methods for...

Add support for LocalDateTime, LocalDate and LocalTime to updateObject()

@marschall or @vlsi are you both satisfied with this ?

Add support for LocalDateTime, LocalDate and LocalTime to updateObject()

@vlsi yes, unless there is anything outstanding in 42.2.2 we should close it off and release

Add support for LocalDateTime, LocalDate and LocalTime to updateObject()

@alexanderkjall can you rebase this and fix the conflicts ?

Add support for LocalDateTime, LocalDate and LocalTime to updateObject()

thx, yes they have

Very rare corrupt timestamps with driver version >= 42.2.25

@melbys Thanks for the report. So as long as we are looking for a proverbial needle in a haystack can you provide any other information? ... is this your own...

Very rare corrupt timestamps with driver version >= 42.2.25

> Are any Connections/ResultSets being processed concurrently? /An/ explanation could be concurrent use of a Calendar object. pretty interesting that the error is always close to the year end.

Very rare corrupt timestamps with driver version >= 42.2.25

Also note that in the two exemplars the hours are close to the TZ offset. I'm liking the Calendar idea. Possibly not fully set before using it ?

Newblobimplementation

@vlsi not pretty but passes all the tests. can you take a look? @bokken I'm curious how much performance we would get by passing in a buffer instead of copying...

feat: Implement Statement#getObject(int, Class) without JSR-310

@marschall thanks for the reminder looks good to me. I'll merge it if there are no other issues ?