kkhatua commented on a change in pull request #1608: DRILL-6960: Auto Limit
Wrapping should not apply to non-select query
URL: https://github.com/apache/drill/pull/1608#discussion_r254118821
##########
File path:
exec/java-exec/src/main/java/org/apache/drill/exec/ops/QueryContext.java
##########
@@ -84,9 +85,14 @@
private boolean closed = false;
private DrillOperatorTable table;
- public QueryContext(final UserSession session, final DrillbitContext
drillbitContext, QueryId queryId) {
+ public QueryContext(UserSession session, DrillbitContext drillbitContext,
QueryId queryId) {
+ this(session, drillbitContext, queryId, null);
+ }
+
+ public QueryContext(UserSession session, DrillbitContext drillbitContext,
QueryId queryId, Integer autoLimit) {
Review comment:
I'm effectively creating an additional constructor with a modified
signature, so as to not change anything else. We cant go the "query option"
solution, so let me see if I can do without it.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
