vvysotskyi commented on a change in pull request #2194:
URL: https://github.com/apache/drill/pull/2194#discussion_r605846389
##########
File path: exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java
##########
@@ -1216,6 +1216,11 @@ public static String bootDefaultFor(String name) {
"the sender to send out its data more rapidly, but you should know
that it has a risk to OOM when the system is solving parallel " +
"large queries until we have a more accurate resource manager."));
+
+ public static final String ENABLE_REST_VERBOSE_ERRORS_KEY =
"rest.errors.verbose";
Review comment:
Thanks, renamed
##########
File path:
exec/java-exec/src/main/java/org/apache/drill/exec/server/options/SystemOptionManager.java
##########
@@ -257,6 +257,7 @@
new OptionDefinition(ClassCompilerSelector.JAVA_COMPILER_JANINO_MAXSIZE),
new OptionDefinition(ClassCompilerSelector.JAVA_COMPILER_DEBUG),
new OptionDefinition(ExecConstants.ENABLE_VERBOSE_ERRORS),
+ new OptionDefinition(ExecConstants.ENABLE_REST_VERBOSE_ERRORS),
Review comment:
Yes, without this step everything will fail.
##########
File path:
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/stream/QueryRunner.java
##########
@@ -48,7 +48,7 @@ public void start(QueryWrapper query) throws
ValidationException {
startQuery(QueryType.valueOf(query.getQueryType()),
query.getQuery(),
userConn);
- userConn.onStart(queryId, maxRows);
+ userConn.onStart(queryId, maxRows, options);
Review comment:
Thanks, used this method
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
