druidry icon indicating copy to clipboard operation
druidry copied to clipboard

Enable ordering for DruidScanQuery

Open pm-src opened this issue 3 years ago • 1 comments

Code change needed:

in.zapr.druid.druidry.query.scan.DruidScanQuery.java

private String order;

@Builder private DruidCustomScanQuery(@NonNull DataSource dataSource, List<DruidVirtualColumn> virtualColumns, .......... ....... Long limit, Boolean legacy, Context context) { . . . this.order = order; if (order != null) { Preconditions.checkArgument((order.equals("ascending") || order.equals("descending")), "only order ascending or descending is supported"); } . . .

}

}

pm-src avatar May 21 '21 01:05 pm-src

Ready for review https://github.com/zapr-oss/druidry/pull/173

szymeker avatar Sep 26 '21 16:09 szymeker