From da17124b34a6995af23b19e809f81cb0ee219bff Mon Sep 17 00:00:00 2001
From: Ernest Mishkin <ernest@returnly.com>
Date: Sun, 10 Dec 2017 18:21:14 -0800
Subject: [PATCH] [#6868] Prefer public constructors

---
 jOOQ/src/main/java/org/jooq/impl/DefaultRecordMapper.java | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/jOOQ/src/main/java/org/jooq/impl/DefaultRecordMapper.java b/jOOQ/src/main/java/org/jooq/impl/DefaultRecordMapper.java
index 4c4959d1e4..ad4b19f4d5 100644
--- a/jOOQ/src/main/java/org/jooq/impl/DefaultRecordMapper.java
+++ b/jOOQ/src/main/java/org/jooq/impl/DefaultRecordMapper.java
@@ -353,6 +353,10 @@ public class DefaultRecordMapper<R extends Record, E> implements RecordMapper<R,
         // "matching" constructor with the same number of fields as this record
         Constructor<E>[] constructors = (Constructor<E>[]) type.getDeclaredConstructors();
 
+        // [#6868] Prefer public constructors
+        Arrays.sort(constructors, (c1, c2) ->
+                (c2.getModifiers() & Modifier.PUBLIC) - (c1.getModifiers() & Modifier.PUBLIC));
+
         // [#1837] If any java.beans.ConstructorProperties annotations are
         // present use those rather than matching constructors by the number of
         // arguments