incubator-s4-user mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Davide Simoncelli <Davide.Simonce...@neclab.eu>
Subject RE: Adapters and S4 cluster
Date Wed, 13 Jun 2012 06:43:55 GMT
I'm sorry. I forgot to check for existing bugs before posting it.

- Davide
________________________________
From: Qiuzhanbo [bohr.qiu@huawei.com]
Sent: Wednesday, June 13, 2012 3:04 AM
To: s4-user@incubator.apache.org
Subject: RE: Adapters and S4 cluster

I have report this bug https://issues.apache.org/jira/browse/S4-51


From: Davide Simoncelli [mailto:Davide.Simoncelli@neclab.eu]
Sent: Tuesday, June 12, 2012 4:59 PM
To: s4-user@incubator.apache.org
Subject: RE: Adapters and S4 cluster


Hello,



I have found a bug on client side implementation when sending keyed events. The method toJson
in Message class converts the message into JSON but it uses a wrong field name for keys' event.
Check here: https://github.com/s4/s4/blob/v0.3.0/s4-driver/java/src/main/java/io/s4/client/Message.java#L87



The documentation at http://docs.s4.io/manual/client_adapter.html#sending-events-into-s4-cluster
says that 'keys' field should be used but the implementation uses 'keysName'. Now keyed events
are dispatched after changing that field to 'keys'.



Should I open a bug ticket for this?



Thank you



- Davide



Mime
View raw message