sdk-java icon indicating copy to clipboard operation
sdk-java copied to clipboard

Why is the header prefix of the Spring module and the Kafka module inconsistent

Open GaryLeiGM opened this issue 4 years ago • 9 comments

The Header prefix in the Spring module is ce-, but the prefix in the kafka module is ce_, so that the Spring module cannot consume the messages produced by the kafka module

GaryLeiGM avatar Mar 15 '21 13:03 GaryLeiGM

The Kafka Protocol binding specification does state that attributes are prefixed with ce_ in binary mode.

Sounds like a potential bug in the Spring module...

JemDay avatar Mar 15 '21 15:03 JemDay

@dsyer is this a bug?

slinkydeveloper avatar Mar 16 '21 17:03 slinkydeveloper

Missing feature maybe.

dsyer avatar Mar 16 '21 17:03 dsyer

Can you tackle it? :smile:

slinkydeveloper avatar Mar 16 '21 17:03 slinkydeveloper

Yeah, sure. I was meaning to do a Kafka sample anyway.

dsyer avatar Mar 16 '21 18:03 dsyer

Any further progress on this? It seems like it's not super straight-forward if it's desirable to only have ce_ for a kafka binder, and retain ce- for other binders.

powturns avatar Aug 05 '21 20:08 powturns

That’s the way the protocol binding is species… Kafka uses ce_ whereas HTTP uses ce-

Jem...

On Aug 5, 2021, at 1:53 PM, Dustin @.***> wrote:

 Any further progress on this? It seems like it's not super straight-forward if it's desirable to only have ce_ for a kafka binder, and retain ce- for other binders.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.

JemDay avatar Aug 05 '21 23:08 JemDay

That’ll teach me to respond to on my iPhone and not spell check…

IPad originated

On Aug 5, 2021, at 4:41 PM, Jem Day @.***> wrote:

That’s the way the protocol binding is species… Kafka uses ce_ whereas HTTP uses ce-

Jem...

On Aug 5, 2021, at 1:53 PM, Dustin @.***> wrote:

 Any further progress on this? It seems like it's not super straight-forward if it's desirable to only have ce_ for a kafka binder, and retain ce- for other binders.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.

JemDay avatar Aug 06 '21 14:08 JemDay