Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

[15:46:47 CDT(-0500)] <chris_cousquer> so it's slow

[15:47:07 CDT(-0500)] <sjungling> @EricDalquist looks like clean deployed & is up an running w/o issue with the modified ehcache.xml

[15:47:14 CDT(-0500)] <EricDalquist> great

[15:48:28 CDT(-0500)] <chris_cousquer> second, this may be have develop to fix & in url string

[15:48:39 CDT(-0500)] <EricDalquist> at this point I'd apply the cache wrappers in personDirectoryContext one at a time

[15:48:43 CDT(-0500)] <EricDalquist> and try

[15:48:47 CDT(-0500)] <EricDalquist> to see if we can narrow this down

[15:48:52 CDT(-0500)] <EricDalquist> since nothing is jumping out at me here

[15:50:56 CDT(-0500)] <EricDalquist> ok I have to run but I'll try to be back online in ~ 40 minutes

[15:51:02 CDT(-0500)] <sjungling> same here

[15:51:04 CDT(-0500)] <EricDalquist> I'll also read the chat logs

[15:51:16 CDT(-0500)] <athena> chris_cousquer: try out the https://github.com/Jasig/NewsReaderPortlet/tree/NEWSPLT-67 branch

[15:51:18 CDT(-0500)] <sjungling> thanks for your help

[15:51:24 CDT(-0500)] <chris_cousquer> so at UPMC we have patch fluid.XMLEncode to return only : return text; and it seems to work for the all portal

[15:51:47 CDT(-0500)] <EricDalquist> no problem sjungling we'll get this figured out (smile)

[15:52:24 CDT(-0500)] <athena> that's good to know, chris_cousquer

[15:55:40 CDT(-0500)] <drewwills> EricDalquist and sjungling... you're probably way ahead of me (i'm catching up on the discussion)... but could there be potential issues with @Autowired on FactoryBean impls? https://jira.springsource.org/browse/SPR-4935

[15:56:19 CDT(-0500)] <drewwills> in my head i can imagine them each trying to do their work in the same magic time

[15:58:22 CDT(-0500)] <chris_cousquer> so I don't know if it's really a bug of fluid or the way we use fluid in newsreaderportlet, athena. I need to do more tests tomorrow and We will try the branch

[15:58:32 CDT(-0500)] <athena> excellent - let me know how it goes

[15:58:44 CDT(-0500)] <athena> i can tell you the title in mobile is sort of broken in that branch still, but i think the rest of it shoudl work

[15:59:12 CDT(-0500)] <chris_cousquer> ok

[16:04:45 CDT(-0500)] <sjungling1> here's something interesting…our OSA found that our Portal apps are querying our LDAP servers for the `guest` user a crazy amount of times… there a way to not search LDAP for the `guest` user?

[16:17:01 CDT(-0500)] <drewwills> yes sjungling1, you could use the RegexGatewayPersonAttributeDao... and also good caching would mitigate that I suppose

[16:20:36 CDT(-0500)] <sjungling1> @drewwills kk. working on the caching right now

[16:22:38 CDT(-0500)] <drewwills> also take a quick look at the IRC logs while you were briefly disconnected: https://wiki.jasig.org/display/UPC/uportal+IRC+Logs

[16:22:54 CDT(-0500)] <drewwills> i posted something, but i think my timing was pretty poor

[16:23:34 CDT(-0500)] <sjungling1> w/r/t @Autowired?

[16:24:05 CDT(-0500)] <drewwills> yes

[16:25:19 CDT(-0500)] <drewwills> i wouldn't call it a promising lead, though... it's not clear to me why that wouldn't be an error all the time, if that hypothesis was correct

[16:25:26 CDT(-0500)] <sjungling1> not sure it was related. think i got my personDirectory working w/ caching…

[16:25:27 CDT(-0500)] <sjungling1> https://gist.github.com/6fca3d482a378f72ef65

[16:25:56 CDT(-0500)] <sjungling1> rolled back to a working config and instead of trying to follow Eric's example, just started uncommenting

[16:26:06 CDT(-0500)] <sjungling1> and that seems to deploy

[16:26:55 CDT(-0500)] <drewwills> ah roger

[16:27:14 CDT(-0500)] <sjungling1> if you could provide a sanity check on that gist, i'd appreciate it

[16:28:09 CDT(-0500)] <sjungling1> as i understand line #80 is calling the cachingMergedPersonAttributeDao which in turn wraps the mergedPersonAttributeDao that I had previously been using?

[16:28:54 CDT(-0500)] <sjungling1> and all of that gets cached under the org.jasig.services.persondir.USER_INFO.merged object?

[16:31:01 CDT(-0500)] <drewwills> wow, you're a pro (wink)

[16:31:20 CDT(-0500)] <sjungling> was i right?

[16:31:24 CDT(-0500)] <drewwills> it's a lot to look at... I can't promise I'm not overlookign something

[16:31:35 CDT(-0500)] <drewwills> yes, that should be accurate

[16:31:49 CDT(-0500)] <drewwills> also nice catch on the cascading impl...

[16:31:54 CDT(-0500)] <sjungling> 'cause, full disclosure, i'm not a Java developer… more of an implementer of Java :-P

[16:32:04 CDT(-0500)] <sjungling> ah… needed Unicon help for that cascading piece

[16:32:26 CDT(-0500)] <drewwills> which you need if you want to chain emplid into cms_student_group, as apparently you do

[16:32:33 CDT(-0500)] <sjungling> bingo

[16:32:59 CDT(-0500)] <drewwills> I remember you, we've met before... I've been on your campus

[16:33:04 CDT(-0500)] <sjungling> guessing that since org.jasig.services.persondir.USER_INFO.merged already exists in ehcache.xml that i don't need to make any mods there

[16:33:27 CDT(-0500)] <drewwills> that should be accurate

[16:33:29 CDT(-0500)] <sjungling> jeez, was that… 7 years ago?

[16:33:51 CDT(-0500)] <drewwills> it might be... i like, helped CSU chico upgrade off 2.x

[16:34:04 CDT(-0500)] <drewwills> maybe even 2.5.x to 2.6.x

[16:34:17 CDT(-0500)] <sjungling> ah yeah… something along those lines

[16:34:33 CDT(-0500)] <drewwills> offices upstairs in the library

[16:35:03 CDT(-0500)] <drewwills> i still get pictures of p.berry's cats on FB

[16:35:10 CDT(-0500)] <sjungling> moved a few years back to the same floor as the data center, but we're slated to move back to the lib next spring

[16:35:24 CDT(-0500)] <sjungling> haha… he was trying to get people to unfollow him but it had the reverse effect

[16:35:32 CDT(-0500)] <drewwills> heh

[16:36:48 CDT(-0500)] <sjungling> alright, pushing out that update personDirectoryContext to our dev server… let's see if i can replicate the magic that was happening on my laptop

[16:47:23 CDT(-0500)] <sjungling> deployed alright. now to try and simulate some traffic