pjfanning commented on code in PR #2632:
URL: https://github.com/apache/drill/pull/2632#discussion_r954255629
##########
exec/java-exec/src/main/resources/drill-module.conf:
##########
@@ -115,7 +115,8 @@ drill.exec: {
text: {
buffer.size: 262144,
batch.size: 4000
- }
+ },
+ recursive_listing_max_size: 10000
Review Comment:
My 2 cents is that limits ideally should be set by default to a sensible
level. For Drill 2.0.0, enforcing that some sort of limit is set would be
something that I'd support. For Drill 1.x, it would not be a good idea to
enforce limits by default but supporting them optionally would be useful (to
avoid introducing changes that might force users to tune configs in a minor
release).
--
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: dev-unsubscr...@drill.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
