cgivre commented on a change in pull request #2096:
URL: https://github.com/apache/drill/pull/2096#discussion_r467897882
##########
File path:
_docs/configure-drill/configuration-options/030-planning-and-exececution-options.md
##########
@@ -27,11 +27,11 @@ You can run the following query to see a list of options:
The query returns a table that lists options with descriptions and other
details. As of Drill 1.15, there are 179 options:
SELECT COUNT() AS num_of_sysopts FROM sys.options;
- +-----------------+
+ |-----------------|
| num_of_sysopts |
- +-----------------+
+ |-----------------|
| 179 |
- +-----------------+
+ |-----------------|
Review comment:
Nit: `###Drill-override.conf` Row Limit Settings is not formatted
correctly.
##########
File path:
_docs/connect-a-data-source/plugins/114-image-metadata-format-plugin.md
##########
@@ -1,6 +1,6 @@
---
title: "Image Metadata Format Plugin"
Review comment:
IMHO, We could use a little re-organization here. This particular page
is about querying image metadata via Drill. We should collect any pages about
formats under the section about querying the file system instead of treating
this like it was a storage plugin.
----------------------------------------------------------------
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]
