-did not mean to get pissy. I know Im new... you are often blunt and it is tempting to respond in kind when I think Im right. -I was assuming that having Scott review my patches was sufficient. My impression of Jboss code review at this point is, post diff and explanation to the list, wait a day or two for comments, and commit. I know Id rather not be the only one to look at a piece of code before committing, but I take it thats pretty common. Its not too clear to me who owns what code, at least within the server - is there a list of people like this? -Is bugzilla being used? Should it be? I know Im keeping a list of bugs that I see posted on the mailing list that I plan to look at. On the mozilla lists, many of the responses to these mails would be: already logged, see (and vote for) bug #10452. But I dont see much of that here. It probably doesnt matter much - jboss volume is much lower. But it might cut down on posts like: hey I found this bug and heres the patch and Im committing now... > -----Original Message----- > From: marc fleury [mailto:[EMAIL PROTECTED]] > Sent: Friday, February 23, 2001 6:58 PM > To: JBoss-Dev; 'Scott M Stark' > Subject: RE: [jBoss-Dev] CVS update: jboss/src/main/org/jboss/ejb > Containe rFactory.java > > > Then it is my turn to explain :) > > We have grown (how many times have I said that in the past > months ;-) alot > and now we have a lot of commits in parallel. JBoss is truly > an open source > group with massive outside contributions. > > Someone was criticizing yesterday that JBoss is now 1000 > commits at the same > time and that 3 people were working at every bit of JBoss. Me? I am > delighted as most of it is major work. > > Someone else was telling me a story about Apache and the day > they understood > they were mature as a public CVS was the day a commit totally > broke down > their production site. Their maturity as CVS and parallel > commits showed > the weakness. > > It had become apparent to them that QA and release was a tough job. I > believe JBoss is in this situation right now and that the > only way to remedy > it is to get the core dedicated to integration with review of commited > patches, constant testing and documentation. That is a full > time job, i > intend to do part of it. > > So I distrust CVS diff messages that go "I changed that (deep > container > structure) and it seems to work for me!!!! whoopee!" approach > to container > development. You *had* done your research , and you *had* > understood the > issues, but you hadn't disclosed it and frankly you are very > new around > here. > > it is about trust among your peers, and don't have the > arrogance to believe > we will give it to you "ab initio" we know too many "bery bery goo > developers"... > > But you did earn this bit of trust, and you did earn a RW > passwd, use it > wisely :) > > marc > > Remember, > *you* are the space monkeys, > *you* are the scum of the earth > -- Fight Club -- > > |-----Original Message----- > |From: [EMAIL PROTECTED] > |[mailto:[EMAIL PROTECTED]]On Behalf Of marc fleury > |Sent: Friday, February 23, 2001 4:27 PM > |To: JBoss-Dev; 'Scott M Stark' > |Subject: RE: [jBoss-Dev] CVS update: jboss/src/main/org/jboss/ejb > |Containe rFactory.java > | > | > |You are absolutely right, > | > |good catch... good pinus. > | > |please commit your change > | > |marc > | > | > ||-----Original Message----- > ||From: [EMAIL PROTECTED] > ||[mailto:[EMAIL PROTECTED]]On Behalf Of Castro, David > ||Sent: Friday, February 23, 2001 2:04 PM > ||To: 'Scott M Stark' > ||Cc: '[EMAIL PROTECTED]' > ||Subject: RE: [jBoss-Dev] CVS update: jboss/src/main/org/jboss/ejb > ||Containe rFactory.java > || > || > ||There are still container specific loaders for every container. > ||The line of > ||code under discussion was creating another application > level classloader > ||which is a duplicate of the one created in J2eeDeployer and which > ||will never > ||be used for anything. > || > ||The container-specific classloaders are created on line 723 of > ||ContainerFactory.java: > || container.setClassLoader( new URLClassLoader( new URL[ 0 > ], cl ) ); > || > ||Currently the classloader stack looks like: > || > ||0. bootstrap loader > || > ||... > || > ||1. Jboss classloader (MLoading: sees the stuff in lib/ext) > || > ||2. App classloader (URLClassLoader: created in > J2eeDeployer: sees ejb-jars > ||and shared libs: delegates to #1: loads all application > classes except the > ||ones in the war file: passed to tomcat and to ContainerFactory) > || > ||3. App classloader (URLClassLoader: created in > ContainerFactory: sees > ||ejb-jars: loads nothing because it delegates to #2.) > || > ||4. Container classloader (URLClassLoader: created in > |ContainerFactory: sees > ||nothing: delegates to #3: used to identify container) > || > ||5. Container local classloader (URLClassLoader: created in > ||ContainerFactory: > ||sees local ejb-jar: does not delegate: loads local xml descriptors) > || > ||6. Tomcat servlet classloader (AdaptiveClassLoader: created by > |Tomcat: sees > ||war file: delegates to #2: loads classes from war file) > || > ||My change was just to eliminate #3, since it doesn't do > anything. We > ||already have classloaders stacked 4-deep, I didn't see any > reason to add > ||another gratuitously. > || > ||Anyway, its not that Important, I just thought Id explain > since people > ||seemed unclear on what was going on. > || > ||dave > || > ||> -----Original Message----- > ||> From: Scott M Stark [mailto:[EMAIL PROTECTED]] > ||> Sent: Friday, February 23, 2001 3:38 PM > ||> To: JBoss-Dev > ||> Cc: Castro, David > ||> Subject: Re: [jBoss-Dev] CVS update: jboss/src/main/org/jboss/ejb > ||> ContainerFactory.java > ||> > ||> > ||> I believe there is a difference in terms of the JNDI > ||> namespace isolation between > ||> containers and so there is utility in the container specific > ||> class loader. I want to > ||> really drill down on the current container/class > ||> loader/namespace relationships > ||> and get this well documented and understood as it affects > ||> many layers. I'm going > ||> to put together something this weekend and submit it for > ||> corrections and additions > ||> next week. > ||> > ||> ----- Original Message ----- > ||> From: "Castro, David" <[EMAIL PROTECTED]> > ||> To: "'JBoss-Dev'" <[EMAIL PROTECTED]> > ||> Sent: Friday, February 23, 2001 12:31 PM > ||> Subject: RE: [jBoss-Dev] CVS update: > ||> jboss/src/main/org/jboss/ejb Containe rFactory.java > ||> > ||> > ||> > It will work this way, too - it just seems pointless to add > ||> yet another > ||> > extraneous layer to the ClassLoader stack. > ||> > > ||> > You know, Marc, I do research and test this stuff before I > ||> send it in. And > ||> > yes, I do understand how the shared classloaders work. > ||> > > ||> > hope that didn't sound too snotty :) > ||> > dave > ||> > > ||> > ||> > || > || > || > || > || > ||------------------------------------------------------------ > --------- > ||This message (including any attachments) contains confidential, > |proprietary > ||or privileged information intended for a specific purpose and > ||individual(s), > ||and is protected by law. If you receive this message in > error, please > ||immediately delete it and all copies of it from your > system, destroy any > ||hard copies of it and notify the sender. Any unauthorized > disclosure, > ||copying or distribution of any part of this message, or the > taking of any > ||unauthorized action based on it, is strictly prohibited. > || > | > | > > --------------------------------------------------------------------- This message (including any attachments) contains confidential, proprietary or privileged information intended for a specific purpose and individual(s), and is protected by law. If you receive this message in error, please immediately delete it and all copies of it from your system, destroy any hard copies of it and notify the sender. Any unauthorized disclosure, copying or distribution of any part of this message, or the taking of any unauthorized action based on it, is strictly prohibited.
