You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2022/04/18 19:01:02 UTC

[GitHub] [druid] techdocsmith commented on a diff in pull request #12088: Clarify the use of the Lookup API

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


##########
docs/querying/lookups.md:
##########
@@ -290,6 +290,8 @@ For example, a post to `/druid/coordinator/v1/lookups/config/realtime_customer1/
 
 This will replace the `site_id_customer1` lookup in the `realtime_customer1` with the definition above.
 
+Make sure to assign a unique version identifier each time you update a lookup extractor factory. If the version already exists, the call will fail.

Review Comment:
   ```suggestion
   Assign a unique version identifier each time you update a lookup extractor factory. Otherwise the call will fail.
   ```



-- 
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-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org