Jackie-Jiang commented on a change in pull request #7784:
URL: https://github.com/apache/pinot/pull/7784#discussion_r752600952
##########
File path:
pinot-clients/pinot-jdbc-client/src/main/java/org/apache/pinot/client/PinotStatement.java
##########
@@ -34,7 +34,7 @@
private ResultSetGroup _resultSetGroup;
private boolean _closed;
private ResultSet _resultSet;
- private int _maxRows = Integer.MAX_VALUE;
+ private int _maxRows = 1000000;
Review comment:
Does the query timeout or throw exception? By default there should be no
limit on this:
```
public static final int DEFAULT_BROKER_QUERY_RESPONSE_LIMIT =
Integer.MAX_VALUE;
```
--
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: commits-unsubscr...@pinot.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
