Jugen commented on code in PR #605:
URL: https://github.com/apache/cayenne/pull/605#discussion_r1485959529


##########
cayenne/src/main/java/org/apache/cayenne/access/translator/select/OrderingStage.java:
##########
@@ -71,4 +85,26 @@ private void processOrdering(QualifierTranslator 
qualifierTranslator, Translator
         context.getSelectBuilder().orderBy(orderingNodeBuilder);
     }
 
+    private Optional<Node> getResultColumn(TranslatorContext context, 
OrderingNodeBuilder orderBuilder) {
+        String orderStr = getSqlString(orderBuilder);
+
+        return context.getResultNodeList().stream()
+                .map(result -> result.getNode())
+                // the column may have an alias so just check that the 
orderStr part matches
+                .filter(resultNode -> 
getSqlString(node(resultNode)).startsWith(orderStr))
+                .findFirst();

Review Comment:
   I thought it's a bit heavy as well. Will need guidance later .....



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@cayenne.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to