add notes

Author: qqmyers

src/main/java/edu/harvard/iq/dataverse/DatasetVersion.java

@@ -216,6 +216,10 @@ public enum VersionState {
     @OneToMany(mappedBy = "datasetVersion", cascade={CascadeType.REMOVE, CascadeType.MERGE, CascadeType.PERSIST})
     private List<WorkflowComment> workflowComments;
 
+    /*
+     * As of v6.7, the NULLS LAST part of the annotation below appears to not be working. Explicit sorting has been added in the getCurationStatuses() method. The annotation is kept since, if it does work
+     * in the future, it is presumably more efficient that sorting in our code.
+     */
     @OneToMany(mappedBy = "datasetVersion", cascade = CascadeType.ALL, orphanRemoval = true)
     @OrderBy("createTime DESC NULLS LAST")
     private List<CurationStatus> curationStatuses = new ArrayList<>();
@@ -2160,6 +2164,11 @@ public String getLocaleLastUpdateTime() {
     // Add methods to manage curationLabels
     public List<CurationStatus> getCurationStatuses() {
         // Sort the list to ensure null createTime values appear last
+        /*
+         * Note that the ORDER DESC NULLS LAST annotation on this list should sort this way,
+         * but, as of v6.7, the NULLS LAST aspect is not working.
+         * The code here assured both the DESC order and NULLS LAST.
+         */
         if (curationStatuses != null) {
             curationStatuses.sort(Comparator.comparing(
                 CurationStatus::getCreateTime,