[I] Docs: Bugs in Kubernetes Quick Start [pinot]

["dttung2905 (via GitHub)" <gi...@apache.org>]

dttung2905 opened a new issue, #12362:
URL: https://github.com/apache/pinot/issues/12362

   Hi team,
   I was following the kuberentes [quickstart docs](https://docs.pinot.apache.org/basics/getting-started/kubernetes-quickstart) and found out a few "Inconsistency"
   -  we don't pin the helm chart of bitnami/kafka to a specific version, and the default value of listener is recently changed to `SASL_PLAINTEXT` from `PLAINTEXT` [source](https://github.com/bitnami/charts/blob/c89408958cef954579eba916c8372bb9ead2a284/bitnami/kafka/README.md?plain=1#L103). We can either pin to a previous helm chart version or use the `LATEST` and update the doc accordingly. Personally, I would go with the latter
   - The path of helm chart is moved to `pinot/helm/pinot` from `pinot/kubernetes/helm/pinot`
   - In the `StreamAvroIntoKafkaCommand` class I found out we set zookeeper address but in the later version of Kafka, Zookeeper address is not needed. Even in the code, I don't think we use it at all though :thinking: please cmiiw https://github.com/apache/pinot/blob/c9a82c40a2c8bed5e86d8278e0bb57bfc5bee86f/pinot-tools/src/main/java/org/apache/pinot/tools/admin/command/StreamAvroIntoKafkaCommand.java#L99-L146
   
   I will make a PR to update the docs and clean up  the `StreamAvroIntoKafkaCommand` if the third point is agreed 
   


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

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


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