<div dir="ltr"><div style="font-size:12.800000190734863px">Hi guys, here&#39;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&#39;ll be producing contain &#39;Variant&#39; (which is an abstract class) we&#39;ll have to come up with a nice solution for that - we&#39;re open to ideas! <br></div><div style="font-size:12.800000190734863px"><br></div><div style="font-size:12.800000190734863px">Right now it won&#39;t work &quot;out of the box&quot; with Polina&#39;s PR due to the List&lt;Variant&gt; 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[&quot;variants&quot;])</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&#39;ll have to make sure our logs are a little more &quot;informative&quot; as we can&#39;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">&lt;<a href="mailto:dimitrazuccarelli@gmail.com" target="_blank">dimitrazuccarelli@gmail.com</a>&gt;</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&#39;s week 11 of GSoC, which means we only have about 3 weeks left until the end! Here&#39;s a quick summary of what we&#39;ve been doing: <div><br></div><div>* Right now I&#39;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&#39;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&#39;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 &quot;custom&quot; producer but hasn&#39;t sent a PR off yet, given the aforementioned issues. She&#39;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&#39;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&#39;ll be working on in our next sprint. </div><div><br></div><div>That&#39;s it, just wanted to update on what&#39;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>