[GitHub] [incubator-druid] gianm commented on a change in pull request #8067: firehose doc adjustments

[GitBox <gi...@apache.org>]

gianm commented on a change in pull request #8067: firehose doc adjustments
URL: https://github.com/apache/incubator-druid/pull/8067#discussion_r303018563
 
 

 ##########
 File path: docs/content/ingestion/firehose.md
 ##########
 @@ -119,10 +118,9 @@ The below configurations can be optionally used for tuning the Firehose performa
 
 ### IngestSegmentFirehose
 
-This Firehose can be used to read the data from existing druid segments.
-It can be used to ingest existing druid segments using a new schema and change the name, dimensions, metrics, rollup, etc. of the segment.
+This Firehose can be used to read the data from existing druid segments, potentially using a new schema and changing the name, dimensions, metrics, rollup, etc. of the segment.
 This Firehose is _splittable_ and can be used by [native parallel index tasks](./native_tasks.html#parallel-index-task).
-A sample ingest Firehose spec is shown below:
+This firehose uses `map` typed parsers. A sample ingest Firehose spec is shown below:
 
 Review comment:
   Does it? The `ingestSegment` firehose example specs in the docs use `string` type parsers.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

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