gnodet commented on PR #26451:
URL: https://github.com/apache/camel/pull/26451#issuecomment-5677522120
Addressed all review findings:
**@apupier / @oscerd / @gnodet-bot** — commits `76ea4c7` and `982cc78`:
1. **Missing regenerated sources (grpc + jt400)** — committed the
annotation-processor output for both:
- `camel-grpc`: `grpc.json` updated with
`forwardOnCompleted`/`forwardOnError` in group `common`; catalog JSON updated
to match
- `camel-jt400`: `Jt400EndpointConfigurer`, `Jt400EndpointUriFactory`,
`jt400.json` regenerated with `outputFieldsIdx`/`fieldsLength` (String)
replacing the old `Integer[]` params; catalog JSON updated to match
- The uncommitted-generated-files gate should now pass
2. **`bootstrapConfiguration` Javadoc** — clarified that the parameter is
URI-resolved by `NettyHttpComponent.createEndpoint()` and is not applied when
set programmatically via the setter
The compile failure (netty-http configurer calling non-existent accessors)
was already fixed in the previous push with the typed
`NettyServerBootstrapConfiguration` field + real getters/setters.
--
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]
