Skip to end of metadata
Go to start of metadata

You are viewing an old version of this page. View the current version.

Compare with Current View Page History

« Previous Version 24 Next »

[08:40:23 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)

[12:11:49 CST(-0600)] <dd> hello

[12:11:54 CST(-0600)] <EricDalquist1> hi

[12:12:19 CST(-0600)] <dd> i'm trying to include clearpass as a dependency

[12:12:33 CST(-0600)] <dd> right now it is listed as a dependency in my uportal-ear/pom.xml file

[12:12:54 CST(-0600)] <athena> what version of uportal are you using?

[12:12:59 CST(-0600)] <dd> when i build, it keeps putting it in shared/lib/clearpass... instead of webapps/uPortal/WEB-INF/lib

[12:13:04 CST(-0600)] <dd> 3/2/4

[12:13:07 CST(-0600)] <dd> 3.2.4

[12:13:40 CST(-0600)] <athena> it needs to be a dependency of uportal-war/pom.xml, rather than the ear

[12:14:51 CST(-0600)] <dd> what's the difference between uportal-impl, uportal-ear, and uportal-war?

[12:14:59 CST(-0600)] <dd> in terms of where to put dependencies

[12:15:31 CST(-0600)] <EricDalquist1> dependencies should go under uportal-impl

[12:15:47 CST(-0600)] <EricDalquist1> impl is all source & libraries

[12:15:52 CST(-0600)] <EricDalquist1> war is webui

[12:16:01 CST(-0600)] <athena> er, right, thanks eric (smile)

[12:16:05 CST(-0600)] <EricDalquist1> ear is a packaging of uPortal and bundled portlets

[12:16:34 CST(-0600)] <EricDalquist1> in 4.0 impl was moved into war

[12:16:36 CST(-0600)] <dd> so if i wanted to do the facebook portlet, it would go in ear

[12:17:28 CST(-0600)] <dd> but a maven overlay would be preferred?

[12:17:33 CST(-0600)] <EricDalquist1> yeah, you add an overlay

[12:17:39 CST(-0600)] <EricDalquist1> and then reference the overlay in uportal-ear

[12:19:33 CST(-0600)] <dd> ok

[12:20:01 CST(-0600)] <dd> one more, Bookmarks Portlet is complaining that it can't find mysql connector

[12:20:21 CST(-0600)] <dd> i see it in uportal/web-inf and weather and webproxy portlet

[12:20:52 CST(-0600)] <dd> can i make it shared or do i need to edit the bookmarks pom.xml to add as a dependency

[12:21:05 CST(-0600)] <EricDalquist1> add it as a bookmarks dep

[12:21:11 CST(-0600)] <EricDalquist1> though it should be getting pulled in by the overlay

[12:21:34 CST(-0600)] <athena> that may not really be configured that way in 3.2

[12:22:02 CST(-0600)] <dd> ok

[12:22:07 CST(-0600)] <dd> thanks, big help

[12:38:22 CST(-0600)] <EricDalquist1> someday I want to refactor uPortal into a bunch of little maven modules

[12:38:42 CST(-0600)] <EricDalquist1> actually have real API seperation internally (tongue)

[12:41:49 CST(-0600)] <athena> that'd be kinda neat

[13:46:12 CST(-0600)] <Arvids> hey, Eric

[13:46:22 CST(-0600)] <EricDalquist1> hi

[13:46:36 CST(-0600)] <Arvids> saw your mail regarding database dialect detection in DAOs

[13:46:55 CST(-0600)] <Arvids> seems like only solution for me... but kinda... dirty

[13:46:58 CST(-0600)] <EricDalquist1> yeah

[13:47:02 CST(-0600)] <EricDalquist1> not sure what our alternative is

[13:47:10 CST(-0600)] <Arvids> on the other hand - hibernate uses that approach a lot

[13:47:12 CST(-0600)] <EricDalquist1> since Postgres absolutely requires an explicit TX to read clobs

[13:47:18 CST(-0600)] <Arvids> indeed

[13:47:35 CST(-0600)] <EricDalquist1> brb

[13:47:36 CST(-0600)] <Arvids> in fact i´m wondering why MSSQL doesn´t require it to be done in transaction

[13:47:57 CST(-0600)] <EricDalquist1> most db's don't

[13:48:02 CST(-0600)] <EricDalquist1> well they do

[13:48:06 CST(-0600)] <EricDalquist1> but they use an implicit tx

[13:48:19 CST(-0600)] <EricDalquist1> which is much nicer from a user point of view

[13:48:30 CST(-0600)] <EricDalquist1> the db knows that reading the row + clob is a two step opperation

[13:48:45 CST(-0600)] <EricDalquist1> and deals with the tx demarcation accordingly to give you a consistent read

[13:49:56 CST(-0600)] <Arvids> ok...

[13:50:14 CST(-0600)] <EricDalquist1> I was concerned about those read-only tx entries

[13:50:21 CST(-0600)] <EricDalquist1> we had something like that before and hit performance issues

[13:52:10 CST(-0600)] <Arvids> sounds like all those places must be reviewed

[13:52:24 CST(-0600)] <EricDalquist1> well the only places they exist right now is around reads of clobs

[13:53:00 CST(-0600)] <EricDalquist1> the other thing I'd like help looking into is why TEXT is failing for mssql

[13:53:07 CST(-0600)] <EricDalquist1> er varchar(max)

[13:53:14 CST(-0600)] <EricDalquist1> since from what I was reading TEXT is deprecated

[13:53:41 CST(-0600)] <dd> hello again

[13:53:50 CST(-0600)] <dd> i'm having problems because of this bug: https://issues.jasig.org/browse/UP-2881?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel#issue-tabs

[13:54:13 CST(-0600)] <dd> is there any way around it to toggle aggregation on/off?

[13:56:41 CST(-0600)] <Arvids> Isn´t that issue fiexed, already?

[13:56:53 CST(-0600)] <dd> it is in 3.2.5, but i am on 3.2.4

[13:57:54 CST(-0600)] <Arvids> i´m not familiar with that feature, but i believe, the right choice would be to patch your portal

[13:58:45 CST(-0600)] <dd> any way to view what changes were made to the relevant files?

[13:59:13 CST(-0600)] <Arvids> ...trying to find them

[13:59:46 CST(-0600)] <Arvids> https://github.com/Jasig/uPortal/commit/f8145b40549d39ca97449ca3e8d3d887066bb417

[14:00:00 CST(-0600)] <Arvids> seems like fix is not that hard at all

[14:00:46 CST(-0600)] <Arvids> btw... is there a way to download a diff file from github?

[14:01:35 CST(-0600)] <dd> nice, thank you

[14:01:46 CST(-0600)] <dd> don't know, i've wondered that too

[14:02:09 CST(-0600)] <Arvids> it would be a nice feature

[14:02:26 CST(-0600)] <EricDalquist1> there is

[14:02:29 CST(-0600)] <EricDalquist1> but it is hidden

[14:02:33 CST(-0600)] <EricDalquist1> add .patch to the commit url

[14:02:34 CST(-0600)] <EricDalquist1> https://github.com/Jasig/uPortal/commit/f8145b40549d39ca97449ca3e8d3d887066bb417.patch

  • No labels