Noticed that this boolean parameter was ignored. (#1041)

After some quick analysis, implemented that way it probably should be used.

Of course, this could have been on purpose, please advise.

src/main/java/io/ebean/text/json/EJson.java

@@ -158,7 +158,12 @@ public class EJson {
     if (list == null) {
       return null;
     }
-    return ((ModifyAwareList) list).asSet();
+
+    if (modifyAware) {
+      return ((ModifyAwareList) list).asSet();
+    } else {
+      return new LinkedHashSet<>(list);
+    }
   }
 
   /**