Added the order by to be applied to the search results even if the order by is part of an aggregation

calebott · calebott · commit 7c5d368dcc61 · 2015-10-19T09:45:23.000-05:00
diff --git a/src/main/java/org/nlpcn/es4sql/query/AggregationQueryAction.java b/src/main/java/org/nlpcn/es4sql/query/AggregationQueryAction.java
@@ -90,6 +90,8 @@ public SqlElasticSearchRequestBuilder explain() throws SqlParseException {
 						break;
 					case "KEY":
 						termsBuilder.order(Terms.Order.term(isASC(order)));
+						// add the sort to the request also so the results get sorted as well
+						request.addSort(order.getName(), SortOrder.valueOf(order.getType()));
 						break;
 					case "FIELD":
 						termsBuilder.order(Terms.Order.aggregation(order.getName(), isASC(order)));
