Jason Parraga
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...
Got some conflicts
> 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....
Keeping this open
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?