Loïc GREFFIER

Results 29 comments of Loïc GREFFIER

Same here, the application is served on _/linked/ui/login_ (not on the default _ui/login_) and I think this is responsible for the bug for us. - First time we log on...

@SouquieresAdam, Good issue, I have considered something like: - Renaming the current [properties](https://github.com/michelin/kstreamplify/blob/6da5112e67d656a67e0ad62e345c216cc9a6cdf6/kstreamplify-core/src/main/java/com/michelin/kstreamplify/context/KafkaStreamsExecutionContext.java#L43) to `kafkaProperties` - In the `KafkaStreamsExecutionContext`, add a new attribute `applicationProperties` containing all other properties that are...

I definitely think that accessing the kafka streams instance should have its own documentation part in the `Getting Started` section as it can be use for multiple purposes, not only...

Initialized by https://github.com/michelin/kstreamplify/pull/204

> Thanks for the PR, some minor comments around code. Thanks for the review @apoorvmittal10. As mentioned, this PR is part of a biggest PR that is being divided (#15973)....

> Thanks for the PR, @loicgreffier! > > Here my feedback! @cadonna The last PR update solved most of the points. I let some conversations open waiting for your feedback

@cadonna Updated! I think only this conversation remains actually open: https://github.com/apache/kafka/pull/16093#discussion_r1648627868

> The e.getMessage() does not contain "Injected test exception" anymore. > > It is now getCause().getMessage() that contains "Injected test exception" > > Fix is pushed @cadonna This is because...

@cadonna @sebastienviale Before this PR, I think processing exceptions are caught and converted to `StreamsException` here: https://github.com/apache/kafka/blob/3ebad6349de7d121a31f9d47c5ede7d6bbfac4d1/streams/src/main/java/org/apache/kafka/streams/processor/internals/StreamTask.java#L805 Thus the uncaught exception handler is receiving a `StreamsException` with the following message:...