May 20, 2011 
< | 1 | 2 | 3 | 4 | 5 | 6 | 7 | 8 | 9 | 10 | 11 | 12 | 13 | 14 | 15 | 16 | 17 | 18 | 19 | 20 | 21 | 22 | 23 | 24 | 25 | 26 | 27 | 28 | 29 | 30 | 31 | >
May 20, 2011 NOTICE: This channel is no longer actively logged.
[00:06:36] *** aslak has quit IRC[00:11:17] *** pferraro has quit IRC[00:11:48] *** asaldhan has quit IRC[00:13:11] *** aslak has joined #jboss-as7[00:13:11] *** ChanServ sets mode: +v aslak[00:16:29] *** pferraro has joined #jboss-as7[00:16:29] *** calvino.freenode.net sets mode: +v pferraro[00:16:57] *** pferraro has quit IRC[00:17:26] *** pferraro has joined #jboss-as7[00:17:26] *** calvino.freenode.net sets mode: +v pferraro[00:21:06] <jbossbot> git [jboss-as] push master 523021a.. bstansberry at jboss dot com Use mvn -fae to ignore test failures[00:21:06] <jbossbot> git [jboss-as] push master URL: http://github.com/jbossas/jboss-as/compare/11b2225...523021a[00:25:23] *** asaldhan has joined #jboss-as7[00:25:50] *** asaldhan has left #jboss-as7[00:32:35] <jamezp> dmlloyd: I'm clearly missing something with Ivy and SVN for IronJacamar. Are there any rules for filing JIRA's on other projects?[00:33:01] <dmlloyd> no but you don't have to do i18n for the IJ project, just file a JIRA[00:33:19] <dmlloyd> if IJ is doing the stupid constructor logger crap then they can deal with the problem[00:33:41] <jamezp> Right, I was just going to file the JIRA for removing the Logger in the constructor.[00:33:46] <jamezp> :-)[00:34:23] *** smcgowan_dinner has quit IRC[00:37:49] *** tcrawley has quit IRC[00:43:44] *** tcrawley has joined #jboss-as7[00:43:44] *** ChanServ sets mode: +v tcrawley[00:43:54] *** bstansberry has quit IRC[00:47:40] *** smarlow has joined #jboss-as7[00:47:41] *** ChanServ sets mode: +v smarlow[00:52:37] *** pferraro has quit IRC[00:53:32] *** smcgowan has joined #jboss-as7[00:53:32] *** ChanServ sets mode: +v smcgowan[01:06:44] <stuartdouglas> dmlloyd: I saw in the logs that we may not actually want that package filtering stuff that I was working on yesterday[01:06:58] <dmlloyd> yeah, something to think about[01:07:04] <stuartdouglas> It is pretty much done, do you want to merge it or just wait and see[01:07:05] <dmlloyd> beacuse we don't have enough to think about![01:07:11] <dmlloyd> hang on to it for a bit[01:07:20] <dmlloyd> maybe we can corner Nihility and discuss it later[01:12:42] *** mbg has joined #jboss-as7[01:13:30] *** lazarotti has quit IRC[01:13:53] *** lazarotti has joined #jboss-as7[01:19:54] <dmlloyd> one thing seems certain though - we should probably have a module which represents all the dependencies that every deployment gets by default[01:20:05] <dmlloyd> javaee.api is nearly that but not quite[01:25:51] *** jc3 has quit IRC[01:28:43] <stuartdouglas> dmlloyd: I think I have sorted out shelly's CDI TCK issues, is there anything in particular you want me to work on now? I was thinking default interceptors[01:34:23] *** jhalliday has joined #jboss-as7[01:34:59] <dmlloyd> default interceptors would be great[01:36:17] *** bbrowning has quit IRC[01:38:06] <stuartdouglas> also now that we have changed the CDI TCK to web profile only 1 test is failing[01:38:17] <stuartdouglas> sorry about that, I was not aware that it had a web profile only mode[01:43:50] *** mbg is now known as mbg|away[01:43:52] *** alexsmirnov has quit IRC[01:44:32] *** sannegrinovero has quit IRC[01:59:25] *** pgier has quit IRC[02:01:11] <jbossbot> git [jboss-as] push master c7c6d9c.. asoldano [AS7-765] support unauthenticated users[02:01:12] <jbossbot> jira [AS7-765] Add JBossWS security-domain before jbossws testsuite runs [Resolved (Done) Task, Major, Alessio Soldano] https://issues.jboss.org/browse/AS7-765[02:01:12] <jbossbot> git [jboss-as] push master 3b8a1ad.. Alexey Loubyansky AS7-786 CLI upload doesn't like Windows paths[02:01:13] <jbossbot> jira [AS7-786] CLI upload doesn't like Windows paths [Open (Unresolved) Bug, Major, Alexey Loubyansky] https://issues.jboss.org/browse/AS7-786[02:01:13] <jbossbot> git [jboss-as] push master a432677.. Scott Marlow AS7-789 JPA default-datasource is optional[02:01:14] <jbossbot> jira [AS7-789] JPA subsystem: empty default DS [Pull Request Sent (Unresolved) Bug, Major, Scott Marlow] https://issues.jboss.org/browse/AS7-789[02:01:14] <jbossbot> git [jboss-as] push master 130094f.. Scott Marlow AS7-806 parse persistence unit (persistence.xml) contained in jar file in the EAR library directory[02:01:15] <jbossbot> jira [AS7-806] parse persistence unit (persistence.xml) contained in jar file in the EAR library directory [Pull Request Sent (Unresolved) Task, Major, Scott Marlow] https://issues.jboss.org/browse/AS7-806[02:01:15] <jbossbot> git [jboss-as] push master df91a0d.. Shelly McGowan AS7-795 - Upgrade to JBossTS 4.15.0[02:01:16] <jbossbot> jira [AS7-795] Upgrade to JBossTS 4.15 [Pull Request Sent (Unresolved) Task, Major, Scott Marlow] https://issues.jboss.org/browse/AS7-795[02:01:16] <jbossbot> git [jboss-as] push master d6e6ad4.. Scott Marlow no longer need TCCL hack with JBossTS 4.15[02:01:16] <jbossbot> git [jboss-as] push master URL: http://github.com/jbossas/jboss-as/compare/523021a...d6e6ad4[02:13:56] *** lazarotti has quit IRC[02:14:24] *** lazarotti has joined #jboss-as7[02:14:42] *** dmison has joined #jboss-as7[02:16:51] *** mbg|away is now known as mbg[02:20:58] *** bgeorges has quit IRC[02:32:01] <dmlloyd> jamezp, it's gonna be impossible to track the used IDs in that doc, I'd just track the allocations there[02:32:15] <dmlloyd> the used IDs should be tracked only in the interface source[02:32:39] <dmlloyd> otherwise you're just going to spend tons of time monkeying with that crappy SBS editr[02:33:01] <jamezp> No problem. So all we need is that simple range then?[02:33:26] <dmlloyd> yeah, just a list of the ranges allocated to each subsystem[02:33:39] <jamezp> That makes it a lot easier :-)[02:33:44] <dmlloyd> it might actually make more sense to list each block of 100 and then put the association after[02:33:58] <dmlloyd> because over time they're going to be non-contiguous and it'd be a pain to search the doc to find the first open block[02:34:42] <jamezp> Yes, that does make more sense. Numerical sort rather than subsystem sort. I like it.[02:38:09] *** JimMa has joined #jboss-as7[02:38:41] <jbossbot> git [jboss-as] push master 5cd99d7.. Paul Gier [AS7-844] Fix Maven warning about deprecated property[02:38:43] <jbossbot> jira [AS7-844] Build updates for productization [Coding In Progress (Unresolved) Task, Major, Paul Gier] https://issues.jboss.org/browse/AS7-844[02:38:43] <jbossbot> git [jboss-as] push master e924b6d.. Paul Gier [AS7-844] Remove duplicate plugin definition from demo2 module...[02:38:43] <jbossbot> git [jboss-as] push master ba77b17.. Paul Gier [AS7-844]Fix typo in assembly plugin config...[02:38:43] <jbossbot> git [jboss-as] push master 245c383.. Paul Gier [AS7-844] Update junit to 4.8.2[02:38:43] <jbossbot> git [jboss-as] push master 1586f76.. Paul Gier [AS7-844] Exclude deprecated artifact sun-jaxb:jaxb-api[02:38:43] <jbossbot> git [jboss-as] push master 75aad86.. Paul Gier [AS7-844] Update antlr 2.7.6-brew -> 2.7.7[02:38:43] <jbossbot> git [jboss-as] push master 9bb25cd.. Paul Gier [AS7-844] Update asm 2.2.3 -> 3.1[02:38:44] <jbossbot> git [jboss-as] push master 113028c.. Paul Gier [AS7-844] Update cglib 2.1_3 -> 2.2[02:38:45] <jbossbot> git [jboss-as] push master 4408577.. Paul Gier [AS7-844] Update gnu-getopt 1.0.12-brew -> 1.0.13[02:38:45] <jbossbot> git [jboss-as] push master URL: http://github.com/jbossas/jboss-as/compare/d6e6ad4...4408577[02:43:54] *** jamezp has quit IRC[02:47:58] *** jamezp has joined #jboss-as7[02:48:39] *** JimMa has quit IRC[02:49:43] <dmlloyd> stuartdouglas: jason will be on shortly to discuss lib filtering[02:49:49] <stuartdouglas> ok[02:50:57] *** Nihility has joined #jboss-as7[02:51:09] *** jamezp is now known as jamezp_afk[02:51:14] *** Nihility has quit IRC[02:51:14] *** Nihility has joined #jboss-as7[02:51:24] <dmlloyd> hey.[02:51:39] <dmlloyd> just bringing up EE class loading requirements to refresh my memory.[02:51:48] <Nihility> Sup[02:53:11] <dmlloyd> okay so.... lorrie and the kids just got home, so I'll be ready in a minute :)[02:53:48] <dmlloyd> quick version: stuart has an impl of auto-filtering ready; we should discuss the alternative of having a single module which contains all default deployment deps[02:53:58] <dmlloyd> and whether we just put all our public APIs in there by default[02:55:04] <stuartdouglas> there are also issues with things like javassist, that are not really a public api as such, but still have to be visible to the deployment[02:55:25] <stuartdouglas> I have seen lots of CDI errors in forums due to javassist being bundled[02:57:45] <dmlloyd> so we have a couple options[02:58:30] <Nihility> I'll bbi a few mins keep talking[02:59:29] <dmlloyd> 1) current solution: detect and filter out certain packages (at deploy time?) from a big list created at build time, auto-replacing with module deps unless configured otherwise[03:00:06] <dmlloyd> 2) just include all public APIs on the deployment module in a parent-first arrangement which prevents bundled libs from being visible, unless configured otherwise[03:00:24] <dmlloyd> 3) do (2) plus filter out the packages to prevent extra classes from "leaking through" to the deployment[03:00:45] <stuartdouglas> I don't think the class leakage is a massive problem[03:00:51] <dmlloyd> one question with 2/3 is whether it would violate the requirements for WP since we'd be exposing EE APIs which may not be actually supported depending on config[03:01:08] <stuartdouglas> as our version of the deps should never reference the unused classes[03:01:41] *** sgilda has quit IRC[03:01:43] *** Nihility has quit IRC[03:02:25] <dmlloyd> haha[03:02:57] *** frainone has joined #jboss-as7[03:02:57] *** ChanServ sets mode: +v frainone[03:03:32] *** lgao has joined #jboss-as7[03:04:57] <stuartdouglas> I doubt it "Web Profile products may support some of the technologies present in the full Java EE Platform and not already listed in Section WP.2.1, “Required Components”, consistently with their compatibility requirements."[03:05:12] <stuartdouglas> from the WP spec[03:19:23] <bobmcw> dmlloyd: if you can hide javaassist somehow, that'd be awesome[03:19:35] <bobmcw> jruby.jar includes javaassist inside, many times relocated using jarjar or whatnot[03:19:40] <bobmcw> but sometimes they don't[03:19:49] <bobmcw> and we hit the javassist version conflict and boom[03:21:41] <stuartdouglas> dmlloyd: Where in the spec does it day that war class loading must be child first?[03:21:50] <stuartdouglas> I can only see this in the servlet spec "It is recommended also that the application class loader be implemented so[03:21:50] <stuartdouglas> that classes and resources packaged within the WAR are loaded in preference to[03:21:51] <stuartdouglas> classes and resources residing in container-wide library JARs"[03:22:00] <stuartdouglas> which suggests it, but does not require it[03:36:00] *** Nihility has joined #jboss-as7[03:36:00] *** Nihility has joined #jboss-as7[03:36:00] *** ChanServ sets mode: +v Nihility[03:39:08] * Nihility backreading[03:43:51] *** mbg is now known as mbg|away[03:45:22] *** mbg|away is now known as mbg[03:55:59] <Nihility> stuartdouglas: i think the ee spec makes it more clear[03:56:22] <Nihility> i dont think we have to worry about exposing an api jar we dont provide an impl for[03:57:48] *** kcbabo has joined #jboss-as7[03:57:48] *** ChanServ sets mode: +v kcbabo[03:58:12] <stuartdouglas> that good[04:00:30] <stuartdouglas> do you know if the war child first class loading is a hard requirement, or just a recommendation?[04:06:44] <dmlloyd> stuartdouglas: right it's basically an assumption because everyone does it that way[04:07:04] <dmlloyd> the spec *also* says that we are free to ignore or supercede any APIs we want[04:07:17] <dmlloyd> or cause deployment to fail at our option[04:07:58] *** JimMa has joined #jboss-as7[04:08:04] *** JimMa has quit IRC[04:09:13] *** stliu has joined #jboss-as7[04:09:13] *** ChanServ sets mode: +v stliu[04:10:16] <Nihility> yeah ultimately it doesnt matter[04:10:55] <dmlloyd> the only reason people did child-first is because they included implementations of crap on the CP by default[04:11:09] <dmlloyd> and in order to allow those to be overridden they pretty much had to allow everything to be overridden[04:11:17] <dmlloyd> with hierarchical models[04:11:32] <dmlloyd> we don't have that problem, so we can do parent-first and have no problems[04:11:38] <stuartdouglas> then how about by default we make war class loading order app server classes, bundled war classes, bundled ear classes[04:11:55] <dmlloyd> sounds fine to me[04:11:57] <stuartdouglas> and allow them to change it in jboss-deployment-structure.xml if they really feel the need[04:12:28] <Nihility> i think we should filter certain things[04:12:33] <Nihility> completely[04:13:26] <dmlloyd> like what?[04:13:35] <stuartdouglas> parent first has basically the same effect[04:13:42] <Nihility> sort of[04:13:50] <dmlloyd> yeah assuming we include all our public stuff in the CP[04:13:56] <Nihility> you can still see resources from the child though[04:14:09] <dmlloyd> true[04:14:20] <dmlloyd> and some things like META-INF stuff can screw things up[04:14:21] <Nihility> which may or may not be a problem[04:14:55] <Nihility> perhaps with api jars[04:14:59] <Nihility> its less of a problem[04:15:29] <stuartdouglas> filtering meta-inf resource is problematic[04:15:41] <stuartdouglas> because of uber-jars, where stuff has been shaded[04:15:56] <Nihility> yeah its hard to know what you can and cant filter[04:16:02] <stuartdouglas> so just because there is one class in the jar that you want to filter, does not mean that you can filter all of them[04:16:03] <Nihility> since there is no assocaition to the package[04:16:21] <Nihility> i was thinking for instance[04:16:26] <Nihility> that javax.servlet[04:16:29] <Nihility> should never be loaded[04:16:32] <Nihility> from a war[04:16:54] <stuartdouglas> yea, but it won't be with parent first[04:16:56] <Nihility> i guess since things are modules[04:17:03] <Nihility> our parent first[04:17:11] <Nihility> filters[04:17:17] <Nihility> compared to herarchical[04:17:35] <Nihility> since you cant split packages[04:19:17] *** smcgowan has quit IRC[04:28:42] *** JimMa has joined #jboss-as7[04:42:03] *** smarlow has quit IRC[04:48:29] <stuartdouglas> I was just thinking that we build the module kinda late in the deployment process, and it could actually be better to build the module as soon as all the processors that deal with dependencies have run[04:49:20] <stuartdouglas> which would mean that in a lot of cases we could start working with Classes and Methods straight away, instead of dealing with the string representations first[04:49:32] <stuartdouglas> not sure how much of a difference it would make though[04:59:17] *** lazarotti has quit IRC[05:05:03] <dmlloyd> well the phases are designed around class loading so really what that amounts to is moving all the other processors later[05:08:00] *** magesh has joined #jboss-as7[05:09:31] *** miclorb_ has quit IRC[05:10:05] *** miclorb has joined #jboss-as7[05:12:01] *** lazarotti has joined #jboss-as7[05:13:59] *** pferraro has joined #jboss-as7[05:13:59] *** ChanServ sets mode: +v pferraro[05:14:05] *** pferraro has quit IRC[05:39:24] *** sguilhen has left #jboss-as7[05:44:48] *** kcbabo has quit IRC[05:45:15] *** miclorb has quit IRC[05:45:50] *** miclorb has joined #jboss-as7[05:53:38] *** lazarotti has quit IRC[05:58:07] *** liweinan has joined #jboss-as7[06:07:56] <jhalliday> are the declared deps in module.xml files just from a classloading perspective, or do they cause lifecycle deps too?[06:08:09] <stuartdouglas> classloading[06:16:07] *** bgeorges has joined #jboss-as7[06:16:07] *** ChanServ sets mode: +v bgeorges[06:16:13] *** miclorb has quit IRC[06:17:15] *** mlinhard has joined #jboss-as7[06:17:54] <jhalliday> good[06:19:53] *** jamezp_afk has quit IRC[06:20:40] <dmlloyd> rule of thumb is modules = CL, MSC = services/lifecycle[06:20:44] <dmlloyd> orthogonal[06:22:06] <jhalliday> well the good news is plugging in the hornetq objectstore seems to need around 3 lines of change. the bad news is allowing it to be configurable is a whole other problem.[06:22:20] <misty> dmlloyd: just curious, do you ever sleep?[06:23:14] <jhalliday> isn't that mutually exclusive with having kids?[06:23:37] <misty> wouldn't know :)[06:24:05] <dmlloyd> a little, from time to time[06:24:11] <dmlloyd> once I start hallucinating[06:24:31] <misty> lol[06:25:45] *** frainone has quit IRC[06:28:46] *** jamezp has joined #jboss-as7[06:42:28] *** Nihility has quit IRC[06:48:31] *** misty has quit IRC[06:56:26] *** jamezp has left #jboss-as7[07:37:56] *** ALR has quit IRC[07:42:14] *** tdiesler has joined #jboss-as7[07:42:21] *** ChanServ sets mode: +v tdiesler[07:47:37] *** JimMa has quit IRC[07:49:24] *** liweinan has quit IRC[07:50:19] *** jhalliday has quit IRC[07:50:22] *** liweinan has joined #jboss-as7[07:53:06] *** JimMa has joined #jboss-as7[08:01:54] *** opalka has joined #jboss-as7[08:01:54] *** ChanServ sets mode: +v opalka[08:02:12] <opalka> morning[08:02:18] <stuartdouglas> morning[08:02:30] <dmlloyd> morning[08:02:50] <stuartdouglas> wow, you really don't sleep[08:13:05] *** tdiesler has quit IRC[08:13:42] *** jfclere has joined #jboss-as7[08:13:42] *** ChanServ sets mode: +v jfclere[08:23:28] <opalka> dmlloyd, WOW[08:23:54] *** tdiesler has joined #jboss-as7[08:23:54] *** ChanServ sets mode: +v tdiesler[08:24:02] *** opalka has quit IRC[08:24:49] <dmlloyd> ?[08:25:02] <dmlloyd> tdiesler: bgeorges was looking for you...[08:25:37] <tdiesler> bgeorges, hi bruno - you wanted something?[08:25:54] *** mgoldmann has joined #jboss-as7[08:25:54] *** ChanServ sets mode: +v mgoldmann[08:36:31] <stuartdouglas> who can merge pull requests for jboss-metadata?[08:37:51] *** tdiesler has quit IRC[08:38:05] *** tdiesler has joined #jboss-as7[08:38:05] *** ChanServ sets mode: +v tdiesler[08:46:43] *** liweinan has quit IRC[08:47:13] *** liweinan has joined #jboss-as7[08:47:21] *** tdiesler has quit IRC[08:49:49] *** tdiesler has joined #jboss-as7[08:49:50] *** ChanServ sets mode: +v tdiesler[08:51:38] *** davidbos has joined #jboss-as7[08:58:03] *** _baleyje has joined #jboss-as7[08:58:03] *** ChanServ sets mode: +v _baleyje[08:58:15] *** baileyje has quit IRC[08:58:16] *** _baleyje is now known as baileyje[09:04:21] *** kevinpollet has joined #jboss-as7[09:11:45] *** stliu has quit IRC[09:13:15] *** alesj has joined #jboss-as7[09:14:29] *** dmison has quit IRC[09:14:48] *** wolfc has joined #jboss-as7[09:14:48] *** ChanServ sets mode: +v wolfc[09:14:50] *** galderz has joined #jboss-as7[09:14:50] *** ChanServ sets mode: +v galderz[09:21:06] *** irooskov has joined #jboss-as7[09:21:06] *** ChanServ sets mode: +v irooskov[09:21:10] *** aslak has quit IRC[09:24:25] <wolfc> jfclere: ping[09:25:41] *** pilhuhn has joined #jboss-as7[09:25:41] *** pilhuhn has joined #jboss-as7[09:25:41] *** ChanServ sets mode: +v pilhuhn[09:32:35] *** emmanuel has joined #jboss-as7[09:32:36] *** ChanServ sets mode: +v emmanuel[09:33:35] *** JimMa has quit IRC[09:34:15] <alesj> wolfc: any idea where is PicketLink source code?[09:34:17] <alesj> which repo[09:34:38] *** tdiesler has quit IRC[09:34:59] <wolfc> alesj: https://svn.jboss.org/repos/picketlink/ ?[09:36:13] <alesj> ah, thanks[09:36:24] *** tdiesler has joined #jboss-as7[09:36:24] *** ChanServ sets mode: +v tdiesler[09:36:26] <alesj> it isn't listed here?[09:36:36] <alesj> http://anonsvn.jboss.org/repos/index.old.html[09:36:39] <alesj> or am i just blind[09:36:59] <alesj> wolfc: ^[09:37:12] <wolfc> I don't see it there either.[09:37:37] <wolfc> Maybe there is a reason it is called index.old :-)[09:37:38] <alesj> yup, strange ...[09:37:46] <alesj> could be[09:37:55] <alesj> or we just don't wanna publicly expose Anil's stuff :-)[09:38:02] <wolfc> :-)[09:38:24] <alesj> imo, should be some criptic url … hehe :-)[09:38:40] <alesj> so only the true daring can find it[09:44:14] <jfclere> wolfc: pong[09:44:39] <jfclere> I was at the gym... Now I am awake :D[09:44:49] <wolfc> jfclere, I see you picked up https://github.com/dmlloyd/metadata/commit/7d497038b667ea6dc8386e23b9db7c1a5517fda2. That breaks bits and pieces.[09:44:49] <jbossbot> git [metadata] 7d49703.. David M. Lloyd Default session bean type to Stateless[09:45:56] <jfclere> Should I roll it back?[09:46:22] <jfclere> What should be the default value?[09:46:27] *** alesj has quit IRC[09:46:54] <wolfc> It should be nothing, because if there is nothing specified in the dd, the annotation should be used.[09:47:05] *** alesj has joined #jboss-as7[09:48:05] *** galderz has quit IRC[09:49:34] <jfclere> wolfc: I will roll it back then[09:49:46] <wolfc> thanks[09:52:57] <jfclere> done[09:56:36] *** tdiesler has quit IRC[09:58:12] *** Jaikiran has joined #jboss-as7[09:58:12] *** ChanServ sets mode: +v Jaikiran[09:58:28] <jfclere> BTW: I have merged Stuart pull request. It should be ok[09:59:11] <stuartdouglas> it was just parsing the interceptor-order element[09:59:23] *** tdiesler has joined #jboss-as7[09:59:23] *** ChanServ sets mode: +v tdiesler[10:01:42] <jfclere> yep adding missing usually don't break other things ;-)[10:02:19] *** aslak has joined #jboss-as7[10:02:21] *** aslak has quit IRC[10:02:21] *** aslak has joined #jboss-as7[10:02:21] *** ChanServ sets mode: +v aslak[10:03:00] *** asoldano has joined #jboss-as7[10:03:01] *** ChanServ sets mode: +v asoldano[10:04:24] <wolfc> stuartdouglas: we're still missing the merging of dd into descriptions. Jaikiran, what's the plan on that?[10:04:47] <stuartdouglas> I am looking at doing that for interceptors at the moment[10:05:00] <stuartdouglas> I'm hoping to get something that you can review out tonight[10:05:02] <wolfc> I'm going to create EJBComponentDescriptionFactory which creates the proper description based of both annotation & dd.[10:05:26] <wolfc> stuartdouglas: we have a very simple review process: pass tests => commit :-)[10:05:27] <Jaikiran> wolfc: I'm currently fixing AS7-852, and once that's done, i can take a look too[10:05:28] <jbossbot> jira [AS7-852] No component found for type '...' [Open (Unresolved) Bug, Major, jaikiran pai] https://issues.jboss.org/browse/AS7-852[10:05:44] <Jaikiran> wolfc: btw, you know what went wrong with george?[10:05:52] <Jaikiran> all tests have started failing for tck[10:06:03] <Jaikiran> and i don't even see a server.log[10:06:16] <wolfc> hmm, usually that's a Monday morning problem...[10:06:33] <stuartdouglas> for the interceptors I am just looking at having a DUP that applies the interceptor information to existing descriptions[10:06:53] <stuartdouglas> after all the annotations etc have been read and added to the description[10:07:52] <wolfc> Jaikiran: it looks like AS 7 is hanging during boot[10:08:06] <Jaikiran> hmm, new issue[10:10:57] *** bgeorges has quit IRC[10:17:05] *** aslak has quit IRC[10:17:45] *** aslak has joined #jboss-as7[10:17:45] *** ChanServ sets mode: +v aslak[10:18:50] *** aslak has quit IRC[10:19:23] *** aslak has joined #jboss-as7[10:19:23] *** ChanServ sets mode: +v aslak[10:23:54] *** adietisheim has joined #jboss-as7[10:23:54] *** ChanServ sets mode: +v adietisheim[10:34:47] *** davidbos has quit IRC[10:36:34] *** jcosta has joined #jboss-as7[10:36:35] *** ChanServ sets mode: +v jcosta[10:37:58] *** torben has joined #jboss-as7[10:37:58] *** torben has quit IRC[10:37:58] *** torben has joined #jboss-as7[10:37:58] *** ChanServ sets mode: +v torben[10:51:28] *** torben has quit IRC[10:56:09] *** torben has joined #jboss-as7[10:56:10] *** ChanServ sets mode: +v torben[10:56:22] *** galderz has joined #jboss-as7[10:56:22] *** ChanServ sets mode: +v galderz[10:57:41] *** davidbos has joined #jboss-as7[10:57:57] *** magesh1 has joined #jboss-as7[11:00:14] *** magesh has quit IRC[11:07:47] *** davidbos has quit IRC[11:24:08] *** hardy_ has joined #jboss-as7[11:27:49] *** tdiesler has quit IRC[11:35:40] *** emmanuel has quit IRC[11:37:42] *** sannegrinovero has joined #jboss-as7[11:37:42] *** sannegrinovero has quit IRC[11:37:42] *** sannegrinovero has joined #jboss-as7[11:37:42] *** ChanServ sets mode: +v sannegrinovero[11:37:44] *** tdiesler has joined #jboss-as7[11:37:44] *** ChanServ sets mode: +v tdiesler[11:41:44] *** kkhan has joined #jboss-as7[11:41:44] *** ChanServ sets mode: +v kkhan[11:51:44] *** tdiesler has quit IRC[11:52:03] *** darranl has joined #jboss-as7[11:52:04] *** darranl has joined #jboss-as7[11:52:04] *** ChanServ sets mode: +v darranl[11:55:15] <kkhan> Hi, am trying to get into office in Newcastle but can't find it. Can someone please copy the numbers from this link and mail to my red hat address? http://intranet.corp.redhat.com/ic/intranet/wiki.html?wikicat=Facilities%3aWhere%20We%20Are%3aNewcastle%2c%20UK&master=00508&toggle=00825[11:58:36] <Jaikiran> kkhan: done[11:59:46] *** Binbin has quit IRC[12:02:15] <Jaikiran> asoldano: was Richard able to create an WS testcase in smoke-tests for https://issues.jboss.org/browse/AS7-852?[12:02:17] <jbossbot> jira [AS7-852] No component found for type '...' [Coding In Progress (Unresolved) Bug, Major, jaikiran pai] https://issues.jboss.org/browse/AS7-852[12:03:27] *** galderz has quit IRC[12:04:37] <asoldano> Jaikiran, I don't know, he's offline due to network issues[12:04:53] <Jaikiran> ok[12:05:23] *** tdiesler has joined #jboss-as7[12:05:23] *** ChanServ sets mode: +v tdiesler[12:09:22] *** davidbos has joined #jboss-as7[12:13:02] *** maeste has joined #jboss-as7[12:13:03] *** ChanServ sets mode: +v maeste[12:21:47] *** galderz has joined #jboss-as7[12:21:47] *** ChanServ sets mode: +v galderz[12:23:50] <kkhan> Thanks, am finally in[12:24:04] *** kkhan has quit IRC[12:24:22] *** kkhan has joined #jboss-as7[12:24:22] *** ChanServ sets mode: +v kkhan[12:24:31] *** pmuir has joined #jboss-as7[12:24:31] *** ChanServ sets mode: +v pmuir[12:30:52] *** asoldano is now known as asoldano_lunch[12:44:20] *** kcbabo has joined #jboss-as7[12:44:20] *** ChanServ sets mode: +v kcbabo[12:51:04] *** torben has quit IRC[13:14:33] *** sgilda has joined #jboss-as7[13:22:22] *** maeste is now known as maeste_lunch[13:23:14] *** pmuir has quit IRC[13:25:05] *** bbrowning has joined #jboss-as7[13:25:05] *** ChanServ sets mode: +v bbrowning[13:28:09] *** pmuir has joined #jboss-as7[13:28:09] *** ChanServ sets mode: +v pmuir[13:32:11] *** jcosta has quit IRC[13:35:02] *** opalka has joined #jboss-as7[13:35:02] *** ChanServ sets mode: +v opalka[13:44:46] *** asoldano_lunch is now known as asoldano[13:53:22] *** magesh1 is now known as magesh[13:57:20] *** jwulf has quit IRC[13:58:45] *** frainone has joined #jboss-as7[13:58:45] *** ChanServ sets mode: +v frainone[14:04:09] *** smarlow has joined #jboss-as7[14:04:09] *** ChanServ sets mode: +v smarlow[14:04:17] *** mgoldmann has quit IRC[14:04:38] *** mgoldmann has joined #jboss-as7[14:04:38] *** ChanServ sets mode: +v mgoldmann[14:04:46] *** galderz has quit IRC[14:06:00] *** galderz has joined #jboss-as7[14:06:00] *** ChanServ sets mode: +v galderz[14:08:31] *** Jaikiran is now known as Jaikiran|AFK[14:08:44] *** galderz has quit IRC[14:11:23] *** galderz has joined #jboss-as7[14:11:23] *** ChanServ sets mode: +v galderz[14:19:53] *** smarlow_ has joined #jboss-as7[14:19:54] *** ChanServ sets mode: +v smarlow_[14:22:03] *** maeste_lunch is now known as maeste[14:23:35] <smarlow_> kkhan: Could you please pull https://github.com/scottmarlow/jboss-as/commit/a076ae713b8de3c19e451f5eca2365814822c50f, it should fix a tck test.[14:23:36] <jbossbot> git [jboss-as] a076ae7.. Scott Marlow for nontx TXEM, find needs to detach loaded entities[14:24:11] * smarlow_ or anyone else that can pull changes[14:25:18] <Jaikiran|AFK> kkhan: also these commits please https://github.com/jaikiran/jboss-as/commits/new-ee-framework[14:25:22] *** Jaikiran|AFK is now known as Jaikiran[14:26:33] <asoldano> Jaikiran, see last Richard's pull request[14:26:42] <asoldano> Jaikiran, regarding AS7-852[14:26:44] <jbossbot> jira [AS7-852] No component found for type '...' [Coding In Progress (Unresolved) Bug, Major, jaikiran pai] https://issues.jboss.org/browse/AS7-852[14:27:15] * Jaikiran checks[14:27:59] *** kevinpollet_ has joined #jboss-as7[14:28:00] *** kevinpollet has quit IRC[14:28:00] *** kevinpollet_ is now known as kevinpollet[14:28:03] *** smarlow_ has quit IRC[14:29:14] <Jaikiran> asoldano: was it sent to some mailing list? i don't see it in the github pull request[14:29:30] <asoldano> Jaikiran, it's in the usual pull request mailing list[14:29:43] <Jaikiran> ah ok. i'm not on it[14:29:51] <Jaikiran> i'll wait for it to be merged upstream[14:30:04] <asoldano> Jaikiran, jbossas-pull-requests at lists dot jboss.org; in any case it's https://github.com/ropalka/jboss-as/commit/7dfa1a4a99c0cc346079913b234a84b04e29ad35[14:30:05] <jbossbot> git [jboss-as] 7dfa1a4.. Richard Opalka providing EJB WS integration test case[14:30:15] *** lgao has quit IRC[14:30:19] <Jaikiran> thanks, that'll help[14:31:00] *** AndyTaylor has joined #jboss-as7[14:31:00] *** ChanServ sets mode: +v AndyTaylor[14:34:04] *** mmoyses has joined #jboss-as7[14:34:04] *** ChanServ sets mode: +v mmoyses[14:38:55] *** Jaikiran is now known as Jaikiran|AFK[14:40:10] * pilhuhn is loosing hair[14:58:50] *** Nihility has joined #jboss-as7[14:58:50] *** Nihility has quit IRC[14:58:50] *** Nihility has joined #jboss-as7[14:58:50] *** ChanServ sets mode: +v Nihility[14:58:53] *** jpederse has joined #jboss-as7[14:58:53] *** ChanServ sets mode: +v jpederse[14:58:58] *** darranl is now known as darranl_afk[15:00:07] <Nihility> morning[15:00:30] <pilhuhn> Hey Nihility[15:00:45] <kkhan> smarlow / Jaikiran|AFK : looking[15:01:04] <jpederse> morning[15:01:19] <kkhan> asoldano: Do you want yours?[15:01:24] <pilhuhn> Anyone knows what log levels subsystem=logging,*=*:change-root-log-level accepts?[15:01:29] *** sguilhen has joined #jboss-as7[15:01:53] <asoldano> kkhan, ?[15:02:23] <kkhan> asoldano: Just back from lunch, but I saw you and Jaikiran|AFK talk about https://github.com/ropalka/jboss-as/commit/7dfa1a4a99c0cc346079913b234a84b04e29ad35[15:02:23] <jbossbot> git [jboss-as] 7dfa1a4.. Richard Opalka providing EJB WS integration test case[15:02:27] <Nihility> pilhuhn: it should take all levels[15:02:43] <pilhuhn> with standard names from log4j ?[15:02:54] <pilhuhn> INFO, DEBUG, WARN, ERROR, FATAL , TRACE?[15:02:59] <asoldano> kkhan, yep, that a test Richard prepared to help Jaikiran|AFK with AS7-852[15:03:00] <jbossbot> jira [AS7-852] No component found for type '...' [Coding In Progress (Unresolved) Bug, Major, jaikiran pai] https://issues.jboss.org/browse/AS7-852[15:03:18] <Nihility> pilhuhn: yes thats correct.[15:03:26] <pilhuhn> Nihility thanks[15:03:43] <Nihility> thunderbird is giving me the spinning wheel of death[15:03:45] <asoldano> kkhan, it should fail (right opalka ?) because of AS7-852[15:03:46] <jbossbot> jira [AS7-852] No component found for type '...' [Coding In Progress (Unresolved) Bug, Major, jaikiran pai] https://issues.jboss.org/browse/AS7-852[15:03:49] <Nihility> is that because its the end of the world today[15:03:50] <Nihility> ?[15:04:31] <opalka> asoldano, kkhan The test is expected to fail[15:04:43] <opalka> asoldano, kkhan but no because of AS7_852 now[15:05:08] <asoldano> opalka, probably a war based example would fail because of that only[15:05:24] <opalka> asoldano, kkhan sorry, didn't test WebServiceContext injection, just enriched the test[15:05:47] <opalka> asoldano, kkhan it can fail because of AS7_852 (but I didn't give it a try - just enriched the test)[15:06:01] <pilhuhn> opalka Hey[15:06:29] <opalka> pilhuhn, hi[15:06:59] <pilhuhn> Are the .jar and .war you sent indepentend or do I need to deploy both?[15:07:34] <pilhuhn> I think when I deploy the .jar the server already tells me it deploys an endpoint and on http://localhost:8080/jaxws-samples-jaxbintros/EndpointService I also get a response[15:07:47] <pilhuhn> but [domain@localhost:9999 subsystem=webservices] :read-resource does not return much[15:08:57] <opalka> pilhuhn, domain usecase doesn't work[15:09:05] <opalka> pilhuhn, I just fixed standalone one[15:09:09] <pilhuhn> ah okay[15:09:17] <pilhuhn> this explains something :->[15:09:20] <opalka> pilhuhn, both jars I provided U are independent (can be deployed both or one by one)[15:09:41] <kkhan> smarlow: branch?[15:09:47] <kkhan> for your commit?[15:10:35] <smarlow> kkhan: its on my master branch. https://github.com/scottmarlow/jboss-as/commit/a076ae713b8de3c19e451f5eca2365814822c50f[15:10:35] <jbossbot> git [jboss-as] a076ae7.. Scott Marlow for nontx TXEM, find needs to detach loaded entities[15:10:43] <kkhan> thanks[15:10:58] <smarlow> kkhan: thank you![15:11:02] <kkhan> smarlow: Do you want all the commits, or just that one?[15:11:36] <smarlow> just that one, I need to move the tests to a different branch until I get past the deploy errors[15:11:52] <kkhan> ok, I'll cherry pick that one[15:14:30] <kkhan> opalka: Building everything now but if the test fails (and is meant to fail) I'll @Ignore it in a follow-up commit. Sound ok?[15:14:50] <opalka> kkhan, it's in integration tests[15:15:01] <opalka> kkhan, those are run only when all test are run[15:15:06] <opalka> kkhan, smoke tests should pass[15:15:39] <opalka> kkhan, but fine with me to @Ignore it :)[15:15:48] *** asaldhan has joined #jboss-as7[15:15:49] *** ChanServ sets mode: +v asaldhan[15:15:51] <kkhan> opalka: I'm doing a -DskipTests=false ;-)[15:17:09] *** fnasser has joined #jboss-as7[15:20:22] <stuartdouglas> can some one merge https://github.com/stuartwdouglas/jboss-as/commits/jpa ?[15:20:33] <stuartdouglas> it's just a fix for the JPA integration tests[15:24:37] *** kkhan has quit IRC[15:25:03] *** aloubyansky has joined #jboss-as7[15:25:30] <smarlow> stuartdouglas: I cherry-picked your fix and will put in a request for it to be pulled (if it doesn't get pulled before then :)[15:26:29] *** slaboure has joined #jboss-as7[15:26:32] <stuartdouglas> I also just moved it into that jpa branch on it's own[15:27:12] <asaldhan> sguilhen: the IIOP errors have been figured out. right?[15:27:22] <sguilhen> asaldhan: no[15:27:49] <sguilhen> asaldhan: I told them yesterday I fixed the TCCL usage. I did it locally and the random error is still there[15:28:57] *** epbernard has joined #jboss-as7[15:28:57] *** epbernard is now known as emmanuel[15:28:57] *** ChanServ sets mode: +v emmanuel[15:29:21] *** kkhan has joined #jboss-as7[15:29:21] *** ChanServ sets mode: +v kkhan[15:30:12] <smarlow> stuartdouglas: I sent email for your AS7-847 (https://github.com/stuartwdouglas/jboss-as/commits/jpa) fix to be pulled. So it won't be forgotten :)[15:30:14] <jbossbot> jira [AS7-847] JPA Extended persistence context regression [Open (Unresolved) Task, Major, Scott Marlow] https://issues.jboss.org/browse/AS7-847[15:33:48] *** darranl_afk is now known as darranl[15:34:05] *** mago1 has joined #jboss-as7[15:34:53] *** smcgowan has joined #jboss-as7[15:34:53] *** ChanServ sets mode: +v smcgowan[15:37:40] *** bobmcw has quit IRC[15:40:59] *** opalka has quit IRC[15:41:23] *** mbg has joined #jboss-as7[15:41:23] *** ChanServ sets mode: +v mbg[15:43:23] <jbossbot> git [jboss-as] push master 99c8193.. jaikiran Remove class which is no longer used after EE framework refactoring[15:43:24] <jbossbot> git [jboss-as] push master 3131c89.. jaikiran Implement @Remove functionality for stateful beans[15:43:24] <jbossbot> git [jboss-as] push master 41d8b45.. jaikiran Minor javadoc addition[15:43:24] <jbossbot> git [jboss-as] push master 0d97e9b.. Richard Opalka providing EJB WS integration test case[15:43:24] <jbossbot> git [jboss-as] push master b4799af.. Scott Marlow for nontx TXEM, find needs to detach loaded entities[15:43:24] <jbossbot> git [jboss-as] push master URL: http://github.com/jbossas/jboss-as/compare/4408577...b4799af[15:52:22] *** magesh has quit IRC[15:52:55] *** frainone has quit IRC[15:54:09] <kkhan> opalka / Jaikiran|AFK I did not @Ignore the tests. Since there are a few failures in the integration tests I am assuming these are allowed[15:54:38] <jbossbot> git [jboss-as] push master f1dc0ab.. Stuart Douglas Fix JPA integration test[15:54:38] <jbossbot> git [jboss-as] push master URL: http://github.com/jbossas/jboss-as/compare/b4799af...f1dc0ab[15:54:59] <smarlow> kkhan: Thanks![15:58:06] *** pgier has joined #jboss-as7[15:58:06] *** ChanServ sets mode: +v pgier[15:58:12] <dmlloyd> hello folkses[15:58:12] *** ccrouch has quit IRC[15:58:30] *** ccrouch has joined #jboss-as7[15:58:30] *** ChanServ sets mode: +v ccrouch[15:59:29] *** galderz has quit IRC[16:03:13] *** bstansberry has joined #jboss-as7[16:03:13] *** ChanServ sets mode: +v bstansberry[16:06:41] *** mago1 has left #jboss-as7[16:09:28] *** stansilvert has joined #jboss-as7[16:09:58] *** mbg has quit IRC[16:13:15] *** galderz has joined #jboss-as7[16:13:15] *** ChanServ sets mode: +v galderz[16:17:31] *** clebert has joined #jboss-as7[16:17:31] *** ChanServ sets mode: +v clebert[16:20:52] *** mago1 has joined #jboss-as7[16:23:10] <mago1> Hi All, is there anyone here know how to deploy EJB as OSGi service in AS7? or is it possible to do it in AS7?[16:24:29] *** irooskov has quit IRC[16:27:04] *** tdiesler has quit IRC[16:27:21] <dmlloyd> not possible yet I don't hink[16:27:52] <dmlloyd> if you have some ideas as to how it should work though, feel free to post to the forums[16:28:17] <dmlloyd> for example, what would the configuration look like, how would the integration work etc.[16:29:32] <mago1> the idea is to have modular application based on OSGi but also get the EE features. I read that it's possible in glassfish. It will be great if AS7 will have it later.[16:30:00] *** tdiesler has joined #jboss-as7[16:30:00] *** ChanServ sets mode: +v tdiesler[16:30:03] <mago1> Here is the article for references: http://weblogs.java.net/blog/ss141213/archive/2010/03/30/ejb-osgi-service-demo-eclipsecon[16:30:22] <mago1> ok, I will try to post it on forum later. Thanks for the info![16:34:48] *** stliu has joined #jboss-as7[16:34:48] *** ChanServ sets mode: +v stliu[16:34:51] *** sebersole has joined #jboss-as7[16:34:51] *** ChanServ sets mode: +v sebersole[16:37:29] *** aloubyansky has quit IRC[16:58:08] *** alexsmirnov has joined #jboss-as7[16:58:26] <Nihility> sguilhen: hey did you replace that sun class though?[17:00:35] *** kcbabo is now known as babo_afk[17:01:03] *** pferraro has joined #jboss-as7[17:01:03] *** ChanServ sets mode: +v pferraro[17:02:03] *** mago1 has left #jboss-as7[17:08:21] *** kevinpollet_ has joined #jboss-as7[17:08:21] *** kevinpollet has quit IRC[17:08:21] *** kevinpollet_ is now known as kevinpollet[17:19:21] *** jfclere has quit IRC[17:21:08] <Nihility> smarlow: did the TX stuff get merged[17:21:25] <smarlow> Nihility: yup, its in[17:24:27] <Nihility> https://github.com/jbossas/jboss-as/pull/6[17:24:32] <Nihility> is that still relevant[17:24:43] *** tdiesler has quit IRC[17:25:12] *** jamezp has joined #jboss-as7[17:27:25] <jpederse> Nihility: ObjectStoreBrowser is management related, better have them rebase[17:27:54] <jpederse> Nihility: you can work on our pull request instead ;)[17:28:23] *** kevinpollet_ has joined #jboss-as7[17:28:25] *** kevinpollet has quit IRC[17:28:25] *** kevinpollet_ is now known as kevinpollet[17:28:32] *** liweinan has quit IRC[17:28:33] *** adietisheim has quit IRC[17:29:54] *** liweinan has joined #jboss-as7[17:30:21] *** liweinan has quit IRC[17:31:55] *** adietisheim has joined #jboss-as7[17:31:56] *** ChanServ sets mode: +v adietisheim[17:33:23] *** kevinpollet has quit IRC[17:34:31] <jamezp> dmlloyd: I've been rejected on my IJ bug for the loggers https://issues.jboss.org/browse/JBJCA-582[17:34:32] <jbossbot> jira [JBJCA-582] Remove org.jboss.logging.Logger from constructors [Closed (Rejected) Bug, Major, Jesper Pedersen] https://issues.jboss.org/browse/JBJCA-582[17:36:27] <jamezp> I could make the message logger extend the basic logger which in turn would satisfy the requirement.[17:36:45] <dmlloyd> jamezp: okay, not worth fighting the political battle, just stuff whatever logger in there[17:37:01] <jamezp> Sounds good. Thanks.[17:37:15] <dmlloyd> leave it between the JCA team and TAG[17:37:30] <dmlloyd> not our problem™[17:37:34] <jamezp> :-)[17:38:02] <jpederse> we need the logger from the concrete class[17:38:21] <jpederse> provide an alternative[17:39:23] <dmlloyd> you really don't[17:39:33] <dmlloyd> if the user wants the class name, they should put the class name in the log formatter[17:39:41] <dmlloyd> otherwise logger categories should be by process, not by class[17:40:22] <dmlloyd> then you can use static loggers, and also be able to convert to i18n which is an EAP6 req[17:40:32] <jpederse> yes, because some of them are not static final -- but maybe getClass() could work[17:41:22] *** tdiesler has joined #jboss-as7[17:41:22] *** ChanServ sets mode: +v tdiesler[17:41:29] <Nihility> "We have requirements that each object hierarchy should use the specific logger of the concrete class."[17:41:34] <Nihility> what requirement is that?[17:41:39] *** AndyTaylor has left #jboss-as7[17:41:55] <jpederse> each connection listener needs to log based on instance[17:43:15] <Nihility> where does that come from, backwards compatiblity or something?[17:43:24] *** pmuir has quit IRC[17:44:37] <jpederse> the connection event listener hierarchy is large - a lot of the work is done in the abstract classes - but we need to track which actual instance is doing the work once something explodes[17:45:29] <dmlloyd> so it's not a requirement then[17:45:32] <dmlloyd> it's just how you do it[17:45:47] <Nihility> hmm but does class really give you per instance[17:45:59] <Nihility> i mean if you want per instance dont you need a name or some kind of identifier[17:46:00] *** tdiesler has quit IRC[17:46:14] <jpederse> dmlloyd: I'm looking at it[17:46:30] <jpederse> jamezp: and yes, the AbstractResourceAdapterDeployer is an easy fix[17:46:53] <dmlloyd> loggers per class is fine for trace stuff[17:47:02] <jpederse> Nihility: this is finding an alternative - not saying it is a bug, fix it[17:47:08] <dmlloyd> but loggers per instance is just a performance pig[17:47:18] *** tcrawley has quit IRC[17:47:23] <jamezp> jpederse: I was going to try, but my lack of SVN and Ivy knowledge prevented me from doing anything useful :-)[17:47:23] <jpederse> dmlloyd: I know[17:47:36] <jpederse> jamezp: no worries, we will fix[17:48:07] <jpederse> jamezp: just docs would still be great[17:48:59] <jamezp> jpederse: I promise to start something today :-) I will at least just copy over what I started, just note it will be a work in progress.[17:49:53] <jpederse> dmlloyd: do we still need if (trace) log.tracef() ?[17:50:01] <Nihility> jpederse: yeah im just trying to help find the alternative/compromise, but i needed to understand the reasons first[17:50:37] <jpederse> Nihility: checking... looks like I have ported most of it to use the 'this' pointer already...[17:50:41] <Nihility> jpederse: if you are using the printf style logs its not as much of an overhead, but you still save a method dispatch[17:52:14] <jpederse> nah, there is crap in there...[17:53:18] <jpederse> jamezp: but it looks like it is easy to port[17:54:50] <jamezp> jpederse: Creating documentation now.[17:55:15] <jpederse> jamezp: sounds good, make sure to list what works and what doesn't[17:56:13] <jamezp> jpederse: Will do. Most of it should be working when we release Beta7. Maybe even today if dmlloyd is okay with it.[17:56:34] <jpederse> jamezp: well, Logger in constructor apparently doesn't ;)[17:56:36] *** Nihility has left #jboss-as7[17:56:45] *** Nihility has joined #jboss-as7[17:56:46] *** ChanServ sets mode: +v Nihility[17:56:49] <Nihility> ∑∑∑∑∑[17:56:58] <bstansberry> darranl: this address isn't correct: /host=local/management-interfaces=native-interface:read-resource[17:56:59] <Nihility> kabir is teaching me special chars[17:57:11] <jpederse> æøå[17:57:17] <bstansberry> s/b "management-interface" with no "s"[17:57:17] <jpederse> there you go[17:57:44] <darranl> bstansberry, ok I am in this area now - I will drop the 's'[17:58:09] <jamezp> jpederse: :-)[17:59:12] *** mbg has joined #jboss-as7[17:59:12] *** ChanServ sets mode: +v mbg[17:59:19] * jamezp needs to decide what category to put the logging documentation in.[17:59:46] <dmlloyd> jpederse: as long as you don't do any interpolation in the method call, log.tracef("format", foo, bar); is just as fast as if (trace) ...[18:00:23] *** davidbos has quit IRC[18:01:07] <jpederse> jamezp: ok, here is one - two object hierarchies sharing a logger instance[18:01:52] *** pferraro has quit IRC[18:02:22] <jpederse> jamezp: one creates it - and initializes the other with it[18:02:25] *** pferraro has joined #jboss-as7[18:02:25] *** ChanServ sets mode: +v pferraro[18:02:56] <jpederse> jamezp: through method call - not constructor - if that makes a difference[18:03:37] *** babo_afk is now known as babo[18:03:37] <dmlloyd> honestly I"d stick with a static logger and for trace calls just do: log.tracef("Something happened '%s' in %s", whatever, this);[18:03:39] *** babo is now known as kcbabo[18:03:47] <dmlloyd> use toString() to provide the relevant information[18:03:57] <jpederse> Nihility: ConnectionManager -> Pool -> Multiple SubPools in case you were wondering[18:04:21] <jpederse> dmlloyd: in the subpool I need to know which pool it relates too, and CM[18:04:36] <dmlloyd> sure, you can log that all in toString() though[18:04:58] <dmlloyd> or in the worst case attach an object whose toString does all that[18:05:10] <dmlloyd> to your base class[18:05:13] <dmlloyd> that way it's still a single method call[18:05:51] <jpederse> the three object trees are independant... looking if I can use pointers...[18:06:06] <jpederse> in the toString()[18:06:14] *** kkhan has quit IRC[18:09:22] *** stliu has quit IRC[18:11:12] *** vtunka has quit IRC[18:12:07] *** pilhuhn is now known as pil-afk[18:12:26] <jpederse> do what are actually the requirements for this to work currently ? static Logger in each class ?[18:12:46] <dmlloyd> not necessarily that, but that's the best way[18:12:50] <dmlloyd> the problem is Logger itself[18:13:07] <dmlloyd> i18n doesn't use that class directly, instead you create logger interfaces[18:15:09] <jpederse> but ....init(......., log) is out too, right ?[18:15:18] <jamezp> jpederse: If you want to see some working examples, take a look at my latest commit https://github.com/jamezp/jboss-as/commit/1d75cf92e510d91adf44c7ba9b099635918cd9f5[18:15:20] <jbossbot> git [jboss-as] 1d75cf9.. James Perkins AS7-809 Use logging tool to create messages and loggers. Convert throwables and logs to use the new interfaces.[18:15:21] <jbossbot> jira [AS7-809] Convert modules to use JBoss Logging and message bundles [Open (Unresolved) Task, Major, James Perkins] https://issues.jboss.org/browse/AS7-809[18:16:23] *** slaboure has quit IRC[18:20:47] <jpederse> jamezp: why did you put the debug and trace in there too ? just to have it all there ?[18:21:39] <jamezp> jpederse: Yes, just to keep it all in the same place and limit the number of loggers needed.[18:21:45] <dmlloyd> yeah that's not a requirement, but probably 90% of projects can do that[18:21:49] <jpederse> k[18:22:10] <dmlloyd> only exception is if you start approaching 65535 messages you're going to have JVM issues :)[18:22:29] <dmlloyd> tbh most of the time debug and trace can just use BasicLogger[18:22:33] <jamezp> jpederse: Rethinking it *may* make more sense to extend the BasicLogger on the interface.[18:22:34] <jpederse> :)[18:22:55] <dmlloyd> however you usually want your exceptions to come from here so they can be translated[18:22:56] <jpederse> jamezp: well, put that in the docs[18:23:08] <jpederse> sure[18:23:26] <jpederse> jamezp: plus standard usage[18:23:26] *** jamezp has quit IRC[18:24:56] *** jamezp has joined #jboss-as7[18:25:14] <jamezp> jpederse: Will do. I'm also going to have some examples. So far I only have examples of creating the interfaces, not the usage. https://github.com/jamezp/jboss-logging-example[18:25:27] <jpederse> jamezp: ok[18:26:15] * jamezp likes to see usage examples to help his understanding.[18:26:21] *** mmoyses is now known as mmoyses_[18:31:59] *** adietisheim has quit IRC[18:32:11] *** miclorb has joined #jboss-as7[18:36:40] *** alexsmirnov has quit IRC[18:37:05] *** alexsmirnov has joined #jboss-as7[18:42:22] *** maeste has quit IRC[18:58:17] *** clebert has quit IRC[18:58:41] *** darranl has quit IRC[19:03:12] *** bobmcw has joined #jboss-as7[19:03:12] *** ChanServ sets mode: +v bobmcw[19:03:15] *** galderz1 has joined #jboss-as7[19:03:16] *** galderz has quit IRC[19:04:20] *** galderz has joined #jboss-as7[19:04:20] *** ChanServ sets mode: +v galderz[19:04:20] *** galderz1 has quit IRC[19:04:50] *** frainone has joined #jboss-as7[19:04:50] *** ChanServ sets mode: +v frainone[19:07:59] *** tcrawley has joined #jboss-as7[19:07:59] *** ChanServ sets mode: +v tcrawley[19:11:24] *** asoldano has quit IRC[19:12:03] *** galderz has quit IRC[19:12:09] *** hardy_ has quit IRC[19:14:03] *** mlinhard has quit IRC[19:29:05] *** sebersole has quit IRC[19:30:18] *** alesj has quit IRC[19:35:03] *** slaboure has joined #jboss-as7[19:37:02] *** alexsmirnov has quit IRC[19:37:38] *** alexsmirnov has joined #jboss-as7[19:40:09] <jbossbot> git [jboss-as] push master 78ee9b2.. Carlo de Wolf Running bits from IDE leaves server.log files all over the place[19:40:09] <jbossbot> git [jboss-as] push master URL: http://github.com/jbossas/jboss-as/compare/f1dc0ab...78ee9b2[19:52:08] *** smcgowan has quit IRC[19:53:07] *** mmoyses_ is now known as mmoyses[19:56:49] *** lazarotti has joined #jboss-as7[19:57:14] *** pil-afk has quit IRC[19:58:53] <jbossbot> git [jboss-as] push master 0a8d61e.. bstansberry at jboss dot com Test for 'relative-to' value in correct place in operation[19:58:53] <jbossbot> git [jboss-as] push master URL: http://github.com/jbossas/jboss-as/compare/78ee9b2...0a8d61e[20:04:12] *** lazarotti has quit IRC[20:06:54] *** lazarotti has joined #jboss-as7[20:09:04] <jamezp> Alright jpederse the first draft is up http://community.jboss.org/wiki/JBossLoggingTooling. It's mostly stolen from dmlloyd's requirements page though.[20:10:48] <jpederse> jamezp: ok, thanks[20:26:26] *** opalka has joined #jboss-as7[20:26:26] *** ChanServ sets mode: +v opalka[20:34:59] *** opalka has quit IRC[20:38:03] *** opalka has joined #jboss-as7[20:38:04] *** opalka has joined #jboss-as7[20:38:04] *** ChanServ sets mode: +v opalka[20:39:32] *** opalka has quit IRC[20:39:51] <bstansberry> dmlloyd: looking at NewOperationContextImpl.doCompleteStep(), it's expecting the callback on the OperationTransaction to happen synchronously[20:39:56] *** opalka has joined #jboss-as7[20:39:56] *** ChanServ sets mode: +v opalka[20:40:16] <Nihility> sguilhen: hey saw your reply, we can't depend on com.sun classes on anything since they are only on sun jvms[20:40:51] <sguilhen> Nihility: so what do we do? I can create my own version of the Impl class and instantiate it instead[20:41:16] <bstansberry> it invokes OperationTransactionControl.handlePrepared() and then immediately returns the result[20:41:22] <Nihility> sguilhen: yeah we need to pull those into that spec package, but only if they are GPL + ClassPath licensed[20:41:25] <Nihility> or LGPL[20:41:25] <dmlloyd> bstansberry: yes[20:41:30] <Nihility> or some other compat license[20:41:50] <dmlloyd> bstansberry: in a remote situation, the OperationTransactionControl will send a "ready" message to the client and block until the client response with a "commit/rollback" message[20:42:13] <dmlloyd> responds[20:43:19] <bstansberry> makes sense[20:45:16] <sguilhen> Nihility: the LICENSE contains a GPL license with a CLASSPATH section[20:45:34] <Nihility> sguilhen: ok then we are good[20:45:53] <Nihility> sguilhen: we just have to use that same license with the spec module[20:46:10] <sguilhen> Nihility: I have copied that license when i created the module[20:46:49] <sguilhen> Nihility: what exactly are we moving to the spec project? only the Impls referenced by the javax.rmi classes or the whole com.sun.corba pack?[20:47:02] <dmlloyd> well we should be using jacorb equivalents[20:47:04] <dmlloyd> or writing them[20:47:17] <sguilhen> there aren't jacorb equivalents[20:47:42] <dmlloyd> ok in that case[20:47:55] <dmlloyd> we have to figure: logically speaking, this works when jacorb is endorsed right?[20:48:12] <sguilhen> probably[20:48:22] <dmlloyd> which means there is some transitive set of classes which work with jacorb but do not encompass all of sun CORBA[20:48:42] <dmlloyd> maybe try using tattletale to figure out the class dependencies?[20:48:45] <Nihility> yeah whats happening is jacorb only refers to spec classes[20:48:50] <Nihility> javax.rmi[20:48:57] <sguilhen> thats right[20:49:06] <Nihility> its the impl of javax.rmi that pulls in these sun classes[20:49:20] <Nihility> so jacorb EXTENDS a javax.rmi class[20:49:22] <dmlloyd> yeah and the sun classes in turn link against the jacorb classes when jacorb is endorsed[20:49:25] <Nihility> which will refer to sun classes[20:49:32] <dmlloyd> instead of the sun ones[20:49:36] <Nihility> no[20:49:39] <dmlloyd> so we just need to figure out what classes we're not overlaying[20:49:53] <dmlloyd> sure they do, because in that case jacorb replaces the JDK[20:49:59] <Nihility> not that i see anyway[20:50:01] <dmlloyd> but only for the classes that jacorb implements[20:50:12] <sguilhen> it will only replace the org.omg classes[20:50:31] <dmlloyd> right but the sun classes would (in the endorsed case) link against jacorb[20:50:41] <dmlloyd> the problem here is that they're linking against the sun versions of org.omg[20:50:45] <Nihility> so far what i have seen is[20:50:55] <Nihility> jacorb USES a small pice of sun code[20:51:01] <Nihility> indirectly[20:51:04] <Nihility> via javax.rmi[20:51:07] <dmlloyd> that's what I'm saying[20:51:13] <Nihility> that code does not link to jacorb[20:51:17] <Nihility> in anyway[20:51:17] <dmlloyd> it uses them, assuming they in turn would be referencing jacorb[20:51:24] <dmlloyd> via org.omg[20:51:31] <dmlloyd> but they don't, hence the issue[20:51:44] <dmlloyd> because, afaict, jacorb assumes it'd be endorsed[20:52:02] *** mbg has quit IRC[20:52:11] <dmlloyd> so we need only to copy over the transitive set of sun classes that we actually *need*[20:52:25] <dmlloyd> which shouldn't encompass all of sun corba I should think[20:52:35] <dmlloyd> tattletale might give a clear picture though[20:52:41] <Nihility> no thats not what i am saying[20:52:46] <dmlloyd> not sure if it will analyze the JDK though[20:53:08] <Nihility> the problem is that we moved javax.rmi into a module[20:53:12] <Nihility> but its half moved[20:53:13] <dmlloyd> okay that was my understanding of the problem[20:53:43] <Nihility> since half of the API stuff is in com.sun.*[20:54:09] <Nihility> you could be write though[20:54:12] <Nihility> right though[20:54:17] <Nihility> in that sucking in that stuff[20:54:20] <Nihility> sucks in the whole thing[20:54:40] <dmlloyd> well either way the fix is to copy in the com.sun classes that we need, preferably repackaged under a different package name to avoid "accidents"[20:54:46] <dmlloyd> org.jboss.com.sun...[20:54:57] *** pferraro has quit IRC[20:55:00] <dmlloyd> org.jboss.com.sun.org.apache....[20:55:01] <dmlloyd> :)[20:55:19] <sguilhen> nice package name, very clear :)[20:55:49] <dmlloyd> then if our javax.rmi doesn't refer to com.sun, even if some TCCL thing makes it malfunction the worst we'll get is CNFE/NCDFE[20:55:57] <dmlloyd> instead of half-loading the wrong impl[20:56:17] *** pferraro has joined #jboss-as7[20:56:17] *** ChanServ sets mode: +v pferraro[20:57:18] <sguilhen> thats true. I'll try to figure out what classes we really need, but I suspect we may need to move all of com.sun.corba[20:57:34] <sguilhen> I was taking a look at the impl classes, they import dozens of other com.sun.corba stuff[20:59:04] <dmlloyd> sure but at some point we'll start hitting things for which jacorb has equivalents[20:59:21] <dmlloyd> or things which we can disable[20:59:52] *** jamezp is now known as jamezp_afk[21:00:14] <dmlloyd> or we could simply not load the sun implementation of these things and just return "no impl found" or whatever[21:02:26] <Nihility> sguilhen: dmlloyd is javax.rmi just rmi-iiop right?[21:02:42] <sguilhen> Nihility: yes[21:03:17] <Nihility> so javax.rmi really should just be a client of org.omg right?[21:03:40] <Nihility> at least at some point[21:03:40] <Nihility> hmm[21:03:51] *** sannegrinovero has quit IRC[21:05:43] <Nihility> ok im looking at this stuff[21:05:46] <sguilhen> Nihility: yea, it uses the org.omg classes[21:05:58] <Nihility> StubIORImpl[21:06:17] <Nihility> that probably could just be a standard IOR right?[21:07:15] <sguilhen> no idea, haven't seen this impl[21:07:23] <Nihility> ah interesting[21:07:38] <Nihility> it looks like they were trying to make a portable ior serialization format[21:07:57] <Nihility> (portable across orbs)[21:10:27] <Nihility> ah you know what[21:10:38] <sguilhen> what[21:10:59] <Nihility> sguilhen: luckily it looks like this isnt too deep, alot of it is just logging stuff and exception m apping[21:11:48] <sguilhen> Nihility: I can start moving the main impl classes and resolve each of the imports separately. This may lead to a more simplified version of the com.sun.corba[21:13:03] <sguilhen> there are three of these Impls: PortableRemoteObject, StubImpl and CORBA.Util. I can bring them over and start looking into the deps, changing the package names to org.jboss.com.sun.corba.*[21:15:55] <Nihility> sguilhen: yeah copying it over and stripping is a good approach[21:16:59] <Nihility> hell even if you pulled in half of the sun orb stuff[21:17:04] <Nihility> it doesnt really matter that much[21:17:11] <Nihility> asside from a file size perspective[21:17:18] <Nihility> as long as all the files have class-path in them[21:17:27] <sguilhen> do you think this should go along with the javax.rmi project?[21:17:41] <Nihility> yeah otherwise its useless[21:17:43] <sguilhen> all files that I checked have classpath[21:17:55] <sguilhen> in the heards, I meant[21:17:59] <Nihility> i would just pull it into a package rename[21:18:02] <sguilhen> *headers*[21:18:03] <Nihility> org.jboss.com.sun[21:18:06] <Nihility> or whatever[21:18:11] <sguilhen> ok, I will do that[21:18:36] <Nihility> this actually could be good for us[21:18:41] <Nihility> if we find some bug in this code[21:18:44] <Nihility> we can actually fix it[21:18:46] <Nihility> consistently[21:18:48] <Nihility> on all VMs[21:19:18] <sguilhen> not to mention we would prob run into problems when running on jrockit, for instance. we don't know what kind of dependencies their impl of javax.rmi brings to the table[21:20:47] <Nihility> yeah its kind of funny SE gets in the way of EE alot these days[21:21:18] <Nihility> would be nice if they would have had a smaller SE[21:21:23] <Nihility> for EE providers[21:23:17] *** pferraro has quit IRC[21:43:17] *** miclorb has quit IRC[21:43:36] *** emmanuel has quit IRC[21:48:50] *** clebert has joined #jboss-as7[21:48:50] *** ChanServ sets mode: +v clebert[22:00:27] *** jamezp_afk is now known as jamezp[22:00:40] *** alesj has joined #jboss-as7[22:05:23] *** jpederse has quit IRC[22:08:19] *** Jaikiran|AFK has quit IRC[22:16:21] *** opalka has quit IRC[22:19:26] *** opalka has joined #jboss-as7[22:19:26] *** opalka has joined #jboss-as7[22:19:26] *** ChanServ sets mode: +v opalka[22:22:13] *** pgier has quit IRC[22:25:43] *** kcbabo has quit IRC[22:28:01] *** slaboure has quit IRC[22:28:04] *** wolfc has quit IRC[22:32:54] *** jbosslog has joined #jboss-as7[22:34:31] *** opalka has quit IRC[22:37:30] *** mgoldmann has quit IRC[22:47:33] *** tcrawley has quit IRC[22:48:12] *** bobmcw has quit IRC[23:01:05] *** bobmcw has joined #jboss-as7[23:01:05] *** ChanServ sets mode: +v bobmcw[23:01:05] *** tcrawley has joined #jboss-as7[23:01:05] *** ChanServ sets mode: +v tcrawley[23:03:31] *** sgilda has quit IRC[23:04:17] <dmlloyd> why do we have another log bot in here[23:04:29] <dmlloyd> jbosslog, meet echelog-2. echelog-2 meet jbosslog[23:05:20] <stuartdouglas> morning[23:05:29] <dmlloyd> hey stuartdouglas.[23:05:46] <dmlloyd> it was a pretty quiet night, I don't have any disasters for you this morning :)[23:07:30] <jamezp> Not that it would matter, today is the end of the world in his TZ :-)[23:08:49] <jbossbot> git [jboss-as] push master 8dbd905.. Stefano Maestri Porting to IJ-Beta6.1 With this version AS7 711 is fixed[23:08:49] <jbossbot> git [jboss-as] push master 936b3fc.. bstansberry at jboss dot com [AS7-711] Port ds config change to testsuite domain config[23:08:50] <jbossbot> jira [AS7-711] Make a version attribute (optional) for driver tag instead the current parsed notation [Coding In Progress (Unresolved) Task, Major, Stefano Maestri] https://issues.jboss.org/browse/AS7-711[23:08:51] <jbossbot> git [jboss-as] push master URL: http://github.com/jbossas/jboss-as/compare/0a8d61e...936b3fc[23:09:19] <stuartdouglas> Next Friday I am going to be giving an introduction to AS7 talk at the Brisbane office[23:09:57] <stuartdouglas> and I was wondering if anyone has any slides I can borrow, so I don't spend the whole weekend making up slides?[23:14:31] *** bbrowning has quit IRC[23:18:16] <dmlloyd> might want to ask on the core list[23:19:59] *** mmoyses has quit IRC[23:23:01] <jbossbot> git [jboss-as] push master 05944cd.. Darran Lofthouse [AS7-811] Added to AbsolutePathService registrations within the HostController process....[23:23:01] <jbossbot> jira [AS7-811] Add required path services to HostController process. [Coding In Progress (Unresolved) Task, Major, Darran Lofthouse] https://issues.jboss.org/browse/AS7-811[23:23:02] <jbossbot> git [jboss-as] push master 10c6cc3.. Darran Lofthouse [AS7-862] Name the service container created for the host controller.[23:23:02] <jbossbot> jira [AS7-862] Name the service container created for the host controller. [Open (Unresolved) Task, Major, Darran Lofthouse] https://issues.jboss.org/browse/AS7-862[23:23:02] <jbossbot> git [jboss-as] push master ade01b3.. Darran Lofthouse [AS7-781] Added support for relative-to when defining the keystore location....[23:23:03] <jbossbot> jira [AS7-781] (Domain Management) Add a relative-to attribute for the keystore location. [Coding In Progress (Unresolved) Task, Major, Darran Lofthouse] https://issues.jboss.org/browse/AS7-781[23:23:03] <jbossbot> git [jboss-as] push master 13ae916.. Darran Lofthouse [AS7-865] Drop the 's' from management-interfaces.[23:23:04] <jbossbot> jira [AS7-865] Drop the 's' from management-interfaces [Open (Unresolved) Task, Major, Darran Lofthouse] https://issues.jboss.org/browse/AS7-865[23:23:04] <jbossbot> git [jboss-as] push master URL: http://github.com/jbossas/jboss-as/compare/936b3fc...13ae916[23:24:50] *** jfclere has joined #jboss-as7[23:27:21] *** bstansberry has quit IRC[23:42:36] *** clebert has quit IRC[23:53:45] *** jfclere has quit IRC[23:58:20] *** miclorb_ has joined #jboss-as7[23:58:48] *** stalep has quit IRC