jihoonson commented on a change in pull request #10748:
URL: https://github.com/apache/druid/pull/10748#discussion_r571297272
##########
File path: docs/ingestion/native-batch.md
##########
@@ -718,7 +718,7 @@ that range if there's some stray data with unexpected
timestamps.
|--------|-----------|-------|---------|
|type|The task type, this should always be "index".|none|yes|
|inputFormat|[`inputFormat`](./data-formats.md#input-format) to specify how to
parse input data.|none|yes|
-|appendToExisting|Creates segments as additional shards of the latest version,
effectively appending to the segment set instead of replacing it. The current
limitation is that you can append to any datasources regardless of their
original partitioning scheme, but the appended segments should be partitioned
using the `dynamic` partitionsSpec.|false|no|
+|appendToExisting|Creates segments as additional shards of the latest version,
effectively appending to the segment set instead of replacing it. This means
that you can append to any datasources regardless of the datasources original
partitioning scheme. `appendToExistoing` requires the `dynamic` partitioning
type for the appended segments. `appendToExisting` does not work with
`forceGuaranteedRollup`. Therefore, if you attempt one of the other
partitioning types, such as `hashed` which requires `forceGuranteedRollup`, the
task will fail and log the following error: "forceGuranteedRollup must be set".
|false|no|
Review comment:
I think you want to apply the same change here as well?
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
