<div dir="ltr">point taken<br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Nov 7, 2013 at 5:35 PM, Apostolos Emmanouilidis <span dir="ltr"><<a href="mailto:aemmanou@redhat.com" target="_blank">aemmanou@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><u></u>
<div>
I consider using UUID as a better choice from a design point of view. If two companies are using UPS and they get merged, the data stored in UPS dbs has to be merged as well. If sequences are used, this task will be a big pain due to the conflicts. However it will be much easier if UUIDs are used. In addition UUID are more flexible when scaling. I'm not sure what happens from a performance point of view.<div>
<div class="h5"><br>
<br>
On Thu, 2013-11-07 at 16:16 +0100, Erik Jan de Wit wrote:
<blockquote type="CITE">
Don't get me wrong I know it's good practice to use a surrogate key instead of a natural one, but why does the key need to be a UUID? Seems that it's a bit big and a sequence would do just as nicely. Then we don't need this 'workaround' creating the keys by 'hand'.
</blockquote>
<blockquote type="CITE">
<br>
<br>
</blockquote>
<blockquote type="CITE">
Because the 'workaround' does have a problem, when would we create the id? Hibernate will 'normally' use null as a value for the id to see that it's a new entity and preforms an insert. But maybe hibernate is smart enough, because you could also use primitives for keys although that is not recommended.
</blockquote>
<blockquote type="CITE">
<br>
<br>
</blockquote>
<blockquote type="CITE">
Anyway that is why I suggest let's keep it simple and use a Long for the id
</blockquote>
<blockquote type="CITE">
<br>
<br>
</blockquote>
<blockquote type="CITE">
On 7 Nov,2013, at 14:48 , Matthias Wessendorf <<a href="mailto:matzew@apache.org" target="_blank">matzew@apache.org</a>> wrote:
</blockquote>
<blockquote type="CITE">
<br>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<br>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<br>
<br>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
On Thu, Nov 7, 2013 at 2:40 PM, Lucas Holmquist <<a href="mailto:lholmqui@redhat.com" target="_blank">lholmqui@redhat.com</a>> wrote:
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<br>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
On Nov 7, 2013, at 7:52 AM, Matthias Wessendorf <<a href="mailto:matzew@apache.org" target="_blank">matzew@apache.org</a>> wrote:
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<br>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<br>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<br>
<br>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
On Thu, Nov 7, 2013 at 1:20 PM, Erik Jan de Wit <<a href="mailto:edewit@redhat.com" target="_blank">edewit@redhat.com</a>> wrote:
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
+1 we can do without the hibernate dependency, making the code more portable. One question it does raise, is why are UUID used as keys? And if a class like PushApplication already has a UUID why would it need another UUID for a key?
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<br>
<br>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
primaryKey (database)
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
variant/pushAPP ID (semantic meaning)
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<br>
<br>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
A while ago we decided to NOT use the PK as the variant/pushAPP ID; let me see if I can find the JIRA/thread
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<br>
<br>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
here is where i brought this up
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<a href="http://aerogear-dev.1069024.n5.nabble.com/aerogear-dev-AeroGear-Push-Message-Format-tp2773p2905.html" target="_blank">http://aerogear-dev.1069024.n5.nabble.com/aerogear-dev-AeroGear-Push-Message-Format-tp2773p2905.html</a>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<br>
<br>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<br>
<br>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<a href="https://issues.jboss.org/browse/AGPUSH-86" target="_blank">https://issues.jboss.org/browse/AGPUSH-86</a><br>
<br>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<br>
<br>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<br>
<br>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<br>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<br>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<br>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
On 7 Nov,2013, at 12:38 , Matthias Wessendorf <<a href="mailto:matzew@apache.org" target="_blank">matzew@apache.org</a>> wrote:
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<br>
<br>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
Hello, <br>
<br>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<br>
<br>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
today the PushApplicationID ([1]) and VariantID (iOS example: [2]) are generated w/in the RESTful endpoint class. I'd like to move that into the actual entity - similar to what we today already do with the (master)secret (e.g. [3] or [4]).
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<br>
<br>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
Thoughts?
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<br>
<br>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<br>
<br>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
While on it - I'd like to do similar to the PK all ALL entities... Today we have an (odd) Hibernate dependency (see [5]), simple b/c of JPA being lame and not providing "propper" UUID support.... So idea is to
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
* remove the odd annotation
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
* do the ID generation "by hand" (like we do on the secret (see above...))
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<br>
<br>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
Thoughts?<br>
<br>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<br>
<br>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
Sure, this (both items of this mail) might lead to some more re-usable code (and better tests)
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<br>
<br>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<br>
<br>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
-Matthias
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<br>
<br>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<br>
<br>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
[1] <a href="https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/src/main/java/org/jboss/aerogear/unifiedpush/rest/registry/applications/PushApplicationEndpoint.java#L76" target="_blank">https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/src/main/java/org/jboss/aerogear/unifiedpush/rest/registry/applications/PushApplicationEndpoint.java#L76</a>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
[2] <a href="https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/src/main/java/org/jboss/aerogear/unifiedpush/rest/registry/applications/iOSVariantEndpoint.java#L105" target="_blank">https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/src/main/java/org/jboss/aerogear/unifiedpush/rest/registry/applications/iOSVariantEndpoint.java#L105</a>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
[3] <a href="https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/src/main/java/org/jboss/aerogear/unifiedpush/model/PushApplication.java#L38" target="_blank">https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/src/main/java/org/jboss/aerogear/unifiedpush/model/PushApplication.java#L38</a>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
[4] <a href="https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/src/main/java/org/jboss/aerogear/unifiedpush/model/AbstractVariant.java#L49" target="_blank">https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/src/main/java/org/jboss/aerogear/unifiedpush/model/AbstractVariant.java#L49</a>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
[5] <a href="https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/src/main/java/org/jboss/aerogear/unifiedpush/jpa/PersistentObject.java#L34" target="_blank">https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/src/main/java/org/jboss/aerogear/unifiedpush/jpa/PersistentObject.java#L34</a>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<br>
<br>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
-- <br>
Matthias Wessendorf <br>
<br>
blog: <a href="http://matthiaswessendorf.wordpress.com/" target="_blank">http://matthiaswessendorf.wordpress.com/</a><br>
sessions: <a href="http://www.slideshare.net/mwessendorf" target="_blank">http://www.slideshare.net/mwessendorf</a><br>
twitter: <a href="http://twitter.com/mwessendorf" target="_blank">http://twitter.com/mwessendorf</a>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
_______________________________________________<br>
aerogear-dev mailing list<br>
<a href="mailto:aerogear-dev@lists.jboss.org" target="_blank">aerogear-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<br>
<br>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<blockquote>
<br>
_______________________________________________<br>
aerogear-dev mailing list<br>
<a href="mailto:aerogear-dev@lists.jboss.org" target="_blank">aerogear-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<br>
<br>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
<br>
<br>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
-- <br>
Matthias Wessendorf <br>
<br>
blog: <a href="http://matthiaswessendorf.wordpress.com/" target="_blank">http://matthiaswessendorf.wordpress.com/</a><br>
sessions: <a href="http://www.slideshare.net/mwessendorf" target="_blank">http://www.slideshare.net/mwessendorf</a><br>
twitter: <a href="http://twitter.com/mwessendorf" target="_blank">http://twitter.com/mwessendorf</a>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<blockquote type="CITE">
_______________________________________________<br>
aerogear-dev mailing list<br>
<a href="mailto:aerogear-dev@lists.jboss.org" target="_blank">aerogear-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<br>
<br>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<blockquote>
<br>
_______________________________________________<br>
aerogear-dev mailing list<br>
<a href="mailto:aerogear-dev@lists.jboss.org" target="_blank">aerogear-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a>
</blockquote>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<br>
<br>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
<br>
<br>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
-- <br>
Matthias Wessendorf <br>
<br>
blog: <a href="http://matthiaswessendorf.wordpress.com/" target="_blank">http://matthiaswessendorf.wordpress.com/</a><br>
sessions: <a href="http://www.slideshare.net/mwessendorf" target="_blank">http://www.slideshare.net/mwessendorf</a><br>
twitter: <a href="http://twitter.com/mwessendorf" target="_blank">http://twitter.com/mwessendorf</a>
</blockquote>
</blockquote>
<blockquote type="CITE">
<blockquote type="CITE">
_______________________________________________<br>
aerogear-dev mailing list<br>
<a href="mailto:aerogear-dev@lists.jboss.org" target="_blank">aerogear-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a>
</blockquote>
</blockquote>
<blockquote type="CITE">
<br>
<pre>_______________________________________________
aerogear-dev mailing list
<a href="mailto:aerogear-dev@lists.jboss.org" target="_blank">aerogear-dev@lists.jboss.org</a>
<a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a>
</pre>
</blockquote>
<br>
</div></div></div>
<br>_______________________________________________<br>
aerogear-dev mailing list<br>
<a href="mailto:aerogear-dev@lists.jboss.org">aerogear-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><br></blockquote></div><br><br clear="all"><div><br></div>-- <br>Matthias Wessendorf <br>
<br>blog: <a href="http://matthiaswessendorf.wordpress.com/" target="_blank">http://matthiaswessendorf.wordpress.com/</a><br>sessions: <a href="http://www.slideshare.net/mwessendorf" target="_blank">http://www.slideshare.net/mwessendorf</a><br>
twitter: <a href="http://twitter.com/mwessendorf" target="_blank">http://twitter.com/mwessendorf</a>
</div></div>