wu-sheng commented on code in PR #10448:
URL: https://github.com/apache/skywalking/pull/10448#discussion_r1167562281
##########
oap-server/oal-rt/src/main/java/org/apache/skywalking/oal/rt/parser/SourceColumn.java:
##########
@@ -34,8 +34,10 @@ public class SourceColumn {
private int length;
private String fieldSetter;
private String fieldGetter;
+ private final boolean groupByColInTopN;
- public SourceColumn(String fieldName, String columnName, Class<?> type,
boolean isID, int length) {
+ public SourceColumn(String fieldName, String columnName, Class<?> type,
boolean isID, int length,
+ boolean groupByColInTopN) {
Review Comment:
Is this `Col` meaning `Column`? I think this is duplicated as this class is
column. My proposal was `Cond` for `Condition`.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
