[04:27:14 EST(-0500)] * higmad (n=chatzill@pcit-8752.HIG.SE) has joined ##uportal
[06:54:47 EST(-0500)] * lennard1 (n=sparhk@wsip-98-174-242-39.ph.ph.cox.net) has joined ##uportal
[08:16:06 EST(-0500)] * bszabo (n=bszabo@wsip-98-174-242-39.ph.ph.cox.net) has joined ##uportal
[09:00:11 EST(-0500)] * EricDalquist (n=dalquist@bohemia.doit.wisc.edu) has joined ##uportal
[09:02:52 EST(-0500)] * athena7 (n=athena7@adsl-99-136-251-32.dsl.wlfrct.sbcglobal.net) has joined ##uportal
[09:07:02 EST(-0500)] <athena7> hey EricDalquist - i had a question about your response on the postgres import/export thing
[09:07:10 EST(-0500)] <EricDalquist> ok
[09:07:27 EST(-0500)] <athena7> do we know for sure that there's always transactions available now?
[09:07:48 EST(-0500)] <EricDalquist> we should assume so
[09:07:48 EST(-0500)] <athena7> i was hesitant to do anything transaction-specific in that code since i saw there was a supportsTransactions test
[09:07:59 EST(-0500)] <EricDalquist> I don't think the portal will work right on a db without Tx support
[09:08:10 EST(-0500)] <athena7> ok, thanks
[09:08:17 EST(-0500)] <athena7> i figured that might be the case but i wanted to make sure
[09:08:29 EST(-0500)] <athena7> wonder if we want to keep that test there
[09:10:11 EST(-0500)] <EricDalquist> yeah ... we can probably just hardcode it to true and move the test to the portal startup checks
[09:10:19 EST(-0500)] <EricDalquist> if it fails then fail starting the portal?
[09:10:40 EST(-0500)] <athena7> yeah, probably a good idea if we're counting on it
[09:11:53 EST(-0500)] <athena7> so will the transaction manager allow us to cleanly start a new transaction if there's already one running?
[09:17:43 EST(-0500)] <EricDalquist> yes
[09:18:35 EST(-0500)] <EricDalquist> let me find an example ....
[09:19:30 EST(-0500)] <EricDalquist> so in the code that does the join tests ....
[09:19:34 EST(-0500)] <EricDalquist> you'd inject a PlatformTransactionManager
[09:20:00 EST(-0500)] <EricDalquist> then in the afterPropertiesSet you would setup a local TransactionTemplate like:
[09:20:02 EST(-0500)] <EricDalquist> this.transactionTemplate = new TransactionTemplate(this.transactionManager);
[09:20:02 EST(-0500)] <EricDalquist> this.transactionTemplate.setPropagationBehavior(TransactionTemplate.PROPAGATION_REQUIRES_NEW);
[09:20:02 EST(-0500)] <EricDalquist> this.transactionTemplate.afterPropertiesSet();
[09:20:42 EST(-0500)] <EricDalquist> that propagation behavior setting there ensures that when you do: this.transactionTemplate.execute(TransactionCallback) it will happen in a new transaction
[09:20:58 EST(-0500)] <EricDalquist> handling creating a new connection and everything
[09:21:29 EST(-0500)] <athena7> ok, that makes sense
[09:24:41 EST(-0500)] * apetro (n=apetro@12.164.139.7) has joined ##uportal
[09:24:43 EST(-0500)] <athena7> by the way, did you have any thoughts on the multi-profile stuff?
[09:24:59 EST(-0500)] <EricDalquist> I haven't had a chance to read it yet
[09:25:04 EST(-0500)] <EricDalquist> last week was a bit crazy
[09:25:14 EST(-0500)] <EricDalquist> I had LASIK on Wednesday so I wasn't working much
[09:26:02 EST(-0500)] <athena7> oh! how'd it go?
[09:26:04 EST(-0500)] <athena7> that's cool 
[09:26:10 EST(-0500)] <EricDalquist> good
[09:26:14 EST(-0500)] <EricDalquist> I can still see 
[09:26:17 EST(-0500)] <athena7> i don't think i can even imagine not having contacts
[09:26:18 EST(-0500)] <EricDalquist> actually I can see very well
[09:26:21 EST(-0500)] <athena7> well that's a plus
[09:26:30 EST(-0500)] <EricDalquist> my eyes have been feeling better every day
[09:26:34 EST(-0500)] <athena7> that's awesome
[09:26:35 EST(-0500)] <athena7> congrats!
[09:26:46 EST(-0500)] <EricDalquist> the worst of the discomfort is no worse than having old contacts in
[09:27:12 EST(-0500)] <athena7> very cool
[09:40:00 EST(-0500)] <dstn> in uPortal 3.0.2 when I drag a portlet sometimes the page just entirely reloads and hangs mid-drag. Anyone experienced that before?
[09:40:16 EST(-0500)] <dstn> doesn't seem to be one specific portlet either
[09:48:12 EST(-0500)] <athena7> actually that is odd
[09:48:24 EST(-0500)] <athena7> that's a known problem for portlets that contain document.write statements
[09:48:32 EST(-0500)] <athena7> particularly anything using a google API
[09:48:50 EST(-0500)] <athena7> but i haven't seen it happen with portlets that don't have that kind of javascript
[09:49:35 EST(-0500)] <athena7> EricDalquist: would it be unreasonable to just pull out the code for the individual query test into a new method and mark it @Transactional(propagation=Propagation.REQUIRES_NEW)?
[09:50:07 EST(-0500)] <EricDalquist> well remember aspects usually only get applied for external calls into a class
[09:50:25 EST(-0500)] <EricDalquist> methodA calling methodB won't trigger aspects unless you're using CGLIB to rewrite the actual class
[09:50:28 EST(-0500)] <athena7> oh.
[09:50:35 EST(-0500)] <athena7> crap, yeah, i always do forget about that
[09:50:35 EST(-0500)] <EricDalquist> so you could refactor them out into a separate class
[09:50:41 EST(-0500)] <athena7> repeatedly
[10:14:56 EST(-0500)] <athena7> i don't know why i always forget that about transactions
[10:15:03 EST(-0500)] <athena7> anyway, this all seems to work as expected
[10:15:07 EST(-0500)] <athena7> er, about annotations
[10:16:59 EST(-0500)] <EricDalquist> the fix is workin?
[10:19:02 EST(-0500)] <athena7> yep
[10:19:09 EST(-0500)] <athena7> everything seems fine
[10:19:13 EST(-0500)] <EricDalquist> awesome!
[10:19:19 EST(-0500)] <athena7> cleaning it up now and stripping out the transaction stuff
[10:19:35 EST(-0500)] <athena7> it'll be so nice to actually be able to use all the import/export features!
[10:22:25 EST(-0500)] <EricDalquist> can you apply the patch to trunk and the patches branch?
[10:23:14 EST(-0500)] <athena7> yes
[10:23:42 EST(-0500)] <athena7> are we still copying stuff over with the svn tool or are we just applying fixes separately now?
[10:28:10 EST(-0500)] <EricDalquist> either
[10:28:14 EST(-0500)] <EricDalquist> I use svnmerge
[10:28:20 EST(-0500)] <EricDalquist> but I'm not going to force anyone else to
General
Content
Integrations