In case of multiple constructor candidates give preference to the public one

Speaking of immutable dto classes, there are many cases where there are several non-default constructors in it. Current implementation of PreferredConstructorDiscoverer.Discoverers.DEFAULT can’t resolve these cases without @PersistenceConstructor. Usually this is not a big problem, however when using libraries such as Lombok or Immutables this adds additional inconvenience. These libraries can generate additional private constructors for builders contructor(builder) or copy methods copyOf(self)/with*

The idea is not to return null when several non-default constructors are found, but try to find one public among them (if it is the only public one) and return it.

[AbstractCoderX]

Yes, this can solve the problem. In immutables also that should be available with:

@Value.Immutable
@Value.Style(of = "new", allParameters = true)
class Test {

  @PersistenceConstructor
  Test() {
  }

}

Аlthough this is solved with two additional lines of code, it becomes very annoying when you need to turn this in almost every dto. One of the cool features of immutables is that you can make custom annotations with style settings and reuse them, but you can’t put a constructor that is declared explicitly (and not in the style annotation) there. Here are my considerations.

[mp911de]

Thanks for bringing up this topic. Let me take it to the team so we can discuss your proposal.