Just to be clear, my criticism was for this to be done only in the
H2Database ("simple change"). I would not mind changing this for all
dialects, if we thoroughly regression test this.On Wed, Nov 6, 2019 at 4:46 PM Lukas Eder <[email protected]> wrote: > > > On Wed, Nov 6, 2019 at 3:49 PM Jens Teglhus Møller <[email protected]> > wrote: > >> Hi Lukas >> >> My argument is that H2 as a database supports running with identifiers in >> both uppercase and lowercase mode, but the jooq H2Database implementation >> limits its use to uppercase mode. >> > > I know what you meant, and that's correct, but... > > >> But there is probably a risk that changes like this leaks bugs into the >> generated model and you are the one that has to support all the quirks, so >> you make the calls. >> > > ... that, and I really don't like doing something only for this H2Database > for all the many reasons I've mentioned before. I've thought about avoiding > quoted identifiers on these jOOQ-meta queries in the past. I couldn't find > the relevant issue just now, so I created one: > https://github.com/jOOQ/jOOQ/issues/9511 > > We'll evaluate this in the future, but it will definitely not go into a > patch release. So the earliest would be 3.13.0 > -- You received this message because you are subscribed to the Google Groups "jOOQ User Group" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/jooq-user/CAB4ELO65zMpF83e9aVz5W1XAiAUS9GpYpAhGSM%3DNbdmydJvrVQ%40mail.gmail.com.
