techdocsmith commented on code in PR #16110:
URL: https://github.com/apache/druid/pull/16110#discussion_r1522121851


##########
docs/development/extensions-core/lookups-cached-global.md:
##########
@@ -211,8 +211,8 @@ The remapping values for each globally cached lookup can be 
specified by a JSON
 |Property|Description|Required|Default|
 |--------|-----------|--------|-------|
 |`pollPeriod`|Period between polling for updates|No|0 (only once)|
-|`uri`|URI for the file of interest, specified as a file, hdfs, s3 or gs 
path|No|Use `uriPrefix`|
-|`uriPrefix`|A URI that specifies a directory (or other searchable resource) 
in which to search for files|No|Use `uri`|
+|`uri`|URI for the file of interest. Can be a file, HDFS, S3 or GCS 
path|Either `uri` or `uriPrefix` must be set|None|

Review Comment:
   ```suggestion
   |`uri`|URI for the lookup file. Can be a file, HDFS, S3 or GCS path|Either 
`uri` or `uriPrefix` must be set|None|
   ```
   File of interest? 😬 



##########
docs/development/extensions-core/lookups-cached-global.md:
##########
@@ -211,8 +211,8 @@ The remapping values for each globally cached lookup can be 
specified by a JSON
 |Property|Description|Required|Default|
 |--------|-----------|--------|-------|
 |`pollPeriod`|Period between polling for updates|No|0 (only once)|
-|`uri`|URI for the file of interest, specified as a file, hdfs, s3 or gs 
path|No|Use `uriPrefix`|
-|`uriPrefix`|A URI that specifies a directory (or other searchable resource) 
in which to search for files|No|Use `uri`|
+|`uri`|URI for the file of interest. Can be a file, HDFS, S3 or GCS 
path|Either `uri` or `uriPrefix` must be set|None|
+|`uriPrefix`|A URI prefix that specifies a directory (or other searchable 
resource) in which to search for files|Either `uri` or `uriPrefix` must be 
set|None|

Review Comment:
   ```suggestion
   |`uriPrefix`|A URI prefix that specifies a directory or other searchable 
resource where lookup files are located |Either `uri` or `uriPrefix` must be 
set|None|
   ```



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to