<div dir="ltr"><div style="font-size:12.800000190734863px">Hi guys, here's the PR for the generic ser/des:<a href="https://github.com/matzew/kafka-cdi/pull/17" target="_blank"> https://github.com/matzew/<wbr>kafka-cdi/pull/17</a><br></div><div><div style="font-size:12.800000190734863px"><br></div><div style="font-size:12.800000190734863px">As far as testing goes, there are a few issues with abstract classes obviously, and since most of the objects we'll be producing contain 'Variant' (which is an abstract class) we'll have to come up with a nice solution for that - we're open to ideas! <br></div><div style="font-size:12.800000190734863px"><br></div><div style="font-size:12.800000190734863px">Right now it won't work "out of the box" with Polina's PR due to the List<Variant> in the PushApplication class: </div><div style="font-size:12.800000190734863px"><br></div><div style="font-size:12.800000190734863px"><p style="margin:0px;font-size:11px;line-height:normal;font-family:Menlo;color:rgb(175,173,36)"><span style="font-variant-ligatures:no-common-ligatures">Caused by: com.fasterxml.jackson.<wbr>databind.JsonMappingException: Can not construct instance of org.jboss.aerogear.<wbr>unifiedpush.api.Variant, problem: abstract types either need to be mapped to concrete types, have custom deserializer, or be instantiated with additional type information</span></p><p style="margin:0px;font-size:11px;line-height:normal;font-family:Menlo"><span style="font-variant-ligatures:no-common-ligatures"><font color="#afad24"> at [Source: [B@d4163ef; line: 1, column: 229] </font><font color="#9900ff">(through reference chain: org.jboss.aerogear.<wbr>unifiedpush.api.<wbr>PushApplication["variants"])</font></span></p><p style="margin:0px;font-size:11px;line-height:normal;font-family:Menlo;color:rgb(175,173,36)"><span style="font-variant-ligatures:no-common-ligatures"><br></span></p><p style="margin:0px;font-size:11px;line-height:normal;font-family:Menlo;color:rgb(175,173,36)"><br></p><p style="margin:0px;line-height:normal"><span style="font-variant-ligatures:no-common-ligatures">Two other things: </span></p><p style="margin:0px;line-height:normal"><span style="font-variant-ligatures:no-common-ligatures"><br></span></p><p style="margin:0px;line-height:normal"><span style="font-variant-ligatures:no-common-ligatures">* Bad things will happen if objects of incorrect type are pushed to the same topic. This will definitely have to be addressed by enforcing some basic security (<a href="https://issues.jboss.org/browse/AGPUSH-2109">AGPUSH-2109</a>)</span></p><p style="margin:0px;line-height:normal"><span style="font-variant-ligatures:no-common-ligatures"><br></span></p><p style="margin:0px;line-height:normal"><span style="font-variant-ligatures:no-common-ligatures">* We'll have to make sure our logs are a little more "informative" as we can't log whole objects obviously for readability. Right now, even with just two consumers, the log is quite overwhelming (i.e. consumer configs etc) </span></p></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 8, 2017 at 9:42 PM, Dimitra Zuccarelli <span dir="ltr"><<a href="mailto:dimitrazuccarelli@gmail.com" target="_blank">dimitrazuccarelli@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi guys,<div><br></div><div>It's week 11 of GSoC, which means we only have about 3 weeks left until the end! Here's a quick summary of what we've been doing: <div><br></div><div>* Right now I'm currently working on implementing custom object serialisers and deserialisers to give us the ability to produce whatever we want to our topics. </div><div>It's been a bit of a bumpy ride so far but Matthias just did a <a href="https://github.com/matzew/kafka-cdi/pull/16" target="_blank">PR</a> which resolves some of the issues we were having with generics. We'll hopefully have this sorted by the end of the week, so we can start using it for the producers/consumers/streams that we implement.</div><div><br></div><div>* Polina has already done <a href="https://issues.jboss.org/browse/AGPUSH-2159" target="_blank">AGPUSH-2159</a> related to our first "custom" producer but hasn't sent a PR off yet, given the aforementioned issues. She's also currently working on getting a fancier test environment up for all of our future consumers and producers :) </div><div><br></div><div>* Since we have a very short amount of time left after that, we'll jump right into some more implementation with <a href="https://issues.jboss.org/browse/AGPUSH-2165" target="_blank">AGPUSH-2165</a> and <a href="https://issues.jboss.org/browse/AGPUSH-2166" target="_blank">AGPUSH-<wbr>2166</a>, which we'll be working on in our next sprint. </div><div><br></div><div>That's it, just wanted to update on what's going on. </div><div>Thanks for the feedback and help, we always appreciate it! </div></div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Dimitra</div></font></span></div>
</blockquote></div><br></div>