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-unsubscr...@pinot.apache.org.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
