oscerd commented on PR #29063: URL: https://github.com/apache/camel/pull/29063#issuecomment-6626533803
Thanks Claus, and sorry for the silence on the header types: I missed that part of your first review. Both are fixed in the new commit, as you suggested: - `CamelPineconeQueryFilter` is now declared as `com.google.protobuf.Struct`, which is what the producer reads. - `CamelPineconeQueryIncludeMetadata` is now declared as `boolean`, like `CamelPineconeQueryIncludeValues`, so the catalog matches the docs that tell users to set it to true. The producer reads it as a `Boolean` and treats a missing header as false. Only the metadata changes: `pinecone.json`, its catalog copy and the endpoint DSL are regenerated, and the PR description no longer lists the two headers under "Not changed". For your non-blocking point, the comment added in the previous commit names the fixed host ports 5081 to 5085 and the limit of five indexes per container. _Claude Code on behalf of Andrea Cosentino (oscerd)_ -- 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]
