Results 135 comments of Jason Parraga

> Thanks for the review @pmahindrakar-oss, I just pushed a change to improve the coverage. I'm having some issues running `helm make`: > > > > ``` > > Save...

> It seems the new spark-operator version requires Go >= 1.24.1. > We might need to wait until [PR #6603](https://github.com/flyteorg/flyte/pull/6603) > (Go 1.24 upgrade) gets merged before moving forward here....

It looks like client side metrics would ideally be implemented with the interceptor concept (which I don't think has been added to this library yet): https://cwiki.apache.org/confluence/display/KAFKA/KIP-42%3A+Add+Producer+and+Consumer+Interceptors

Flyte seems to be using pinned docker images so I don't think it will be hard to change the repository to bitnami legacy. It might be a little more challenging...

Looks like go mod tidy needs to be run

@ddl-ebrown do you think we can close this or do you want to retry after the go changes that have been made?

@wildfluss can you sign off your commits?

@wildfluss Can you also run `make helm` in the project root and check in the generated code?