cryptoe commented on code in PR #18530:
URL: https://github.com/apache/druid/pull/18530#discussion_r2386659469


##########
docs/ingestion/data-formats.md:
##########
@@ -1053,6 +1056,17 @@ For example, using Avro Hadoop parser with custom 
reader's schema file:
 
 ### ORC Hadoop Parser
 
+:::caution[Deprecated]

Review Comment:
   I see this config para repeated multiple times. 
   Can we create a single page, capture the contents of the para and have the 
config block explicitly point to the single page.
   That way any changes need to be done only in one place. 



##########
docs/release-info/upgrade-notes.md:
##########
@@ -38,6 +38,85 @@ For more information, see [Migration guide: front-coded 
dictionaries](./migr-fro
 
 If you're already using this feature, you don't need to take any action. 
 
+## 34.0.0
+
+### Upgrade notes
+
+#### Hadoop-based ingestion
+
+Hadoop-based ingestion has been deprecated since Druid 32.0 and will be 
removed as early as Druid 35.0.0. 

Review Comment:
   ```suggestion
   Hadoop-based ingestion has been deprecated since Druid 32.0 and will be 
removed in Druid 37.0.0. 
   ```
   Wdyt ?



-- 
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