...
[08:07:44 CST(-0600)] <EricDalquist> ok I think we can work around this
[08:07:52 CST(-0600)] <EricDalquist> using criteria builder
[08:08:12 CST(-0600)] <EricDalquist> you can essentially tell the query builder explicitly to load certain collections
[08:08:36 CST(-0600)] <Arvids> yeah, but that turns your criteria building a nightmare
[08:08:53 CST(-0600)] <Arvids> lot´s of stuff to tell that you´ve already declared via annotations
[08:09:10 CST(-0600)] <EricDalquist> yeah
[08:26:19 CST(-0600)] <EricDalquist> so making jpa fetch collections is pretty easy
[08:26:45 CST(-0600)] <EricDalquist> for example in JpaStylesheetDescriptorDao I added:
[08:26:46 CST(-0600)] <EricDalquist> descriptorRoot.fetch(StylesheetDescriptorImpl_.layoutAttributes);
[08:26:46 CST(-0600)] <EricDalquist> descriptorRoot.fetch(StylesheetDescriptorImpl_.outputProperties);
[08:26:46 CST(-0600)] <EricDalquist> descriptorRoot.fetch(StylesheetDescriptorImpl_.stylesheetParameters);
[08:27:02 CST(-0600)] <EricDalquist> that then results in a single query to load the descriptor and all three maps
[08:27:29 CST(-0600)] <EricDalquist> really I can't wait for the natural-id loading stuff that I've been helping the hibernate folks with
[08:27:38 CST(-0600)] <EricDalquist> it will really solve a lot of our db related loading issue
[08:27:40 CST(-0600)] <EricDalquist> issues
[08:37:00 CST(-0600)] <Arvids> this part i did figure out, but it retrieved a lot of attribute duplicates... and that was the part where I gave up
[08:40:14 CST(-0600)] <EricDalquist> ah well it always well
[08:40:18 CST(-0600)] <EricDalquist> that is the nature of joins
[08:40:37 CST(-0600)] <EricDalquist> the stuff in the primary entity will be listed in every result row
[08:42:45 CST(-0600)] <Arvids> that irritates me - why do I have to tell JPA same things over and over again? Isn´t that the point of ORM - single point of configuration for each entity?
[08:43:18 CST(-0600)] <Arvids> after all - that´s the reason why we´re using typesafe queries
[08:48:35 CST(-0600)] <EricDalquist> yeah, that is annoying
[08:48:51 CST(-0600)] <EricDalquist> but this does reduce queries even more
[08:49:03 CST(-0600)] <EricDalquist> query for id then load by id can be 2 queries
[08:49:08 CST(-0600)] <EricDalquist> this will be one at most
[08:49:25 CST(-0600)] <Arvids> agree
[08:53:42 CST(-0600)] <Arvids> ... then my last commit should be reverted and previous criteria building changed
[08:54:11 CST(-0600)] <EricDalquist> yeah, no problem
[10:02:27 CST(-0600)] <Arvids> https://github.com/Jasig/uPortal/pull/18
[10:04:34 CST(-0600)] <Arvids> Feel free to suggest any improvements/refactorings needed
[10:05:17 CST(-0600)] <Arvids> tomorrow i´ll try to tinker the stylesheet user preferences code
[10:09:59 CST(-0600)] <EricDalquist> sounds good, I'll take a look shortly
[10:15:06 CST(-0600)] <Arvids> take your time... I don´t want to feel guilty for stealing your day off 
[10:44:56 CST(-0600)] <dd_> hello
[10:45:16 CST(-0600)] <dd_> trying to setup SmartLdapGroupStore
[10:45:24 CST(-0600)] <dd_> i was using PAGS before
[10:45:33 CST(-0600)] <dd_> i followed this guide: https://wiki.jasig.org/display/UPM32/SmartLdapGroupStore
[10:45:49 CST(-0600)] <dd_> logged in, how would i tell if SmartLdapGroupStore is setup?
[13:07:50 CST(-0600)] * Topic is 'http://uportal.pastebin.com/ - https://wiki.jasig.org/display/UPC/uportal+IRC+Logs' set by EricDalquist!~apollo@76.210.64.142 on 17:44:12 CDT(-0500)