[00:06:14] *** jbossbot has joined #seam-dev
[00:30:05] *** clerum has joined #seam-dev
[00:37:51] *** jose_freitas has quit IRC
[00:46:08] *** gastaldi has joined #seam-dev
[00:46:23] *** gastaldi has left #seam-dev
[00:46:35] <jbossbot> git [servlet] push master 02c851c.. LightGuard Fixes SEAMSERVLET-28 (https://issues.jboss.org/browse/SEAMSERVLET-28)...
[00:46:36] <jbossbot> jira [SEAMSERVLET-28] Integrate Catch with *EventBridge* classes [Open (Unresolved) Enhancement, Blocker, Unassigned] https://issues.jboss.org/browse/SEAMSERVLET-28
[00:46:37] <jbossbot> git [servlet] push master URL: http://github.com/seam/servlet/compare/6ddf3fe...02c851c
[00:46:57] *** lightguard_jp has joined #seam-dev
[01:06:07] *** maxandersen has joined #seam-dev
[01:09:10] <jbossbot> git [servlet] push master 6db1fc2.. LightGuard SEAMSERVLET-28 docs
[01:09:11] <jbossbot> jira [SEAMSERVLET-28] Integrate Catch with *EventBridge* classes [Resolved (Done) Enhancement, Blocker, Unassigned] https://issues.jboss.org/browse/SEAMSERVLET-28
[01:09:11] <jbossbot> git [servlet] push master URL: http://github.com/seam/servlet/compare/02c851c...6db1fc2
[01:09:58] *** lightguard_jp has quit IRC
[01:20:39] *** maxandersen has quit IRC
[02:07:11] <jbossbot> git [mail] push master f5eac9d.. Cody Lerum suffle classes into correct packages
[02:07:11] <jbossbot> git [mail] push master 36aa623.. Cody Lerum drop ClassPathEmailAttachment and ClassPathTemplate. Use InputStream...
[02:07:11] <jbossbot> git [mail] push master 5c8c5b4.. Cody Lerum refactor for a more unified templating system and a single MailMessage...
[02:07:11] <jbossbot> git [mail] push master 483122f.. Cody Lerum update dist pom
[02:07:11] <jbossbot> git [mail] push master 06c0bb8.. Cody Lerum refactor for a more unified templating experieince
[02:07:11] <jbossbot> git [mail] push master URL: http://github.com/seam/mail/compare/15140fd...06c0bb8
[02:25:54] *** gastaldi has joined #seam-dev
[02:27:38] *** gastaldi has left #seam-dev
[02:38:06] *** gastaldi has joined #seam-dev
[02:38:22] <gastaldi>  Hi all http://maps.google.com/maps?ll=0.000000,0.000000
[02:38:48] <gastaldi> Ignore this link
[02:39:06] <gastaldi> Just testing a new IRC app on iPhone
[02:41:13] *** gastaldi has left #seam-dev
[04:21:10] *** daniel_hinojosa has quit IRC
[05:23:06] *** clerum has quit IRC
[05:38:30] *** gastaldi has joined #seam-dev
[06:01:08] *** daniel_hinojosa has joined #seam-dev
[06:07:33] *** rruss has joined #seam-dev
[06:07:45] *** rruss has quit IRC
[07:52:57] *** clerum has joined #seam-dev
[08:20:46] *** clerum has quit IRC
[10:24:49] *** daniel_hinojosa has quit IRC
[12:00:03] *** bitshuffler has joined #seam-dev
[12:32:16] *** bitshuffler_ has joined #seam-dev
[12:33:12] *** bitshuffler has quit IRC
[12:47:35] *** bitshuffler__ has joined #seam-dev
[12:50:58] *** bitshuffler_ has quit IRC
[13:01:42] *** bitshuffler has joined #seam-dev
[13:04:50] *** bitshuffler__ has quit IRC
[13:15:56] *** bitshuffler_ has joined #seam-dev
[13:18:59] *** bitshuffler has quit IRC
[13:35:39] *** clerum has joined #seam-dev
[13:44:02] *** mgencur has joined #seam-dev
[13:44:12] *** mgencur has left #seam-dev
[13:50:36] *** maxandersen has joined #seam-dev
[14:03:58] *** maxandersen has quit IRC
[14:05:35] *** gastaldi has joined #seam-dev
[14:05:51] *** gastaldi has left #seam-dev
[15:12:40] *** johnament has joined #seam-dev
[15:31:35] *** jose_freitas has joined #seam-dev
[15:46:03] *** mgencur has joined #seam-dev
[15:46:06] *** mgencur has left #seam-dev
[15:51:26] *** jose_freitas has quit IRC
[15:53:23] *** clerum has quit IRC
[16:27:40] *** jose_freitas has joined #seam-dev
[16:45:38] <johnament> is there an updated seam parent that has CR1 binaries?
[16:47:19] <jose_freitas> bom 9
[16:47:43] <jose_freitas> <parent>
[16:47:44] <jose_freitas>       <groupId>org.jboss.seam</groupId>
[16:47:44] <jose_freitas>       <artifactId>seam-parent</artifactId>
[16:47:44] <jose_freitas>       <version>9</version>
[16:47:44] <jose_freitas>    </parent>
[16:48:46] <johnament> thanks.
[16:48:58] *** gastaldi has joined #seam-dev
[16:49:01] *** lincolnthree has joined #seam-dev
[16:49:11] <gastaldi> Hello all !
[16:49:26] <lincolnthree> *crickets*
[16:49:27] <johnament> jose_freitas: seam.version ?
[16:49:38] <gastaldi> Lol
[16:50:29] <johnament> crickets >> gun shots
[16:51:02] <gastaldi> A Ball of hay
[16:51:58] <gastaldi> Hey johnament ! How'doin ?
[16:52:31] <gastaldi> Dos you manage to reproduce that issue in JCR also ?
[16:52:52] <gastaldi> Dos/ did
[16:53:19] <gastaldi> Stupid iPhone autosuggestion feature
[16:54:01] <lincolnthree> you mean awesome autosuggestion future
[16:55:05] <johnament> gastaldi: it's not specific to any one module.
[16:55:12] <johnament> gastaldi: i can reproduce it in a really simple application.
[16:55:47] <gastaldi> Yeah, I understand that it might be a weld issue
[16:56:57] <gastaldi> But I did not understand why those tests passed, because I thought that arquillian just uses weld
[16:57:22] <johnament> we're all scratching our heads on that
[16:57:44] <johnament> similar, in Seam JMS we were able to get bean objects created BEFORE the container initialized them, using arquillian
[16:58:35] <johnament> compare the 1.1 docs http://docs.jboss.org/weld/reference/1.1.0.Final/en-US/html_single/#d0e3895
[16:58:46] <johnament> to the 1.0.1 docs http://docs.jboss.org/weld/reference/1.0.1-Final/en-US/html_single/#d0e4044
[16:58:57] <gastaldi> Are you using An equivalent code in JMS ? Even selecting before firing ?
[16:59:27] <johnament> event selecting just returns a new event instance with the extra qualifiers.
[16:59:50] <johnament> the issue has to do with how observer methods are chosen.
[17:01:14] <gastaldi> Does it envolve observes without any qualifiers ?
[17:02:26] <gastaldi> I understand that you said that observes w/o qualifiers are called even when specyfing one
[17:04:47] <johnament> gastaldi: here's the example app https://github.com/johnament/RestSending
[17:04:57] <gastaldi> I
[17:05:38] <johnament> gastaldi: in this case, you can change line 33, which should change what observes get notified.
[17:05:48] <johnament> in SendEventResource.java
[17:08:23] <johnament> in all cases though, you'll see that all observer methods are notified.
[17:09:11] <johnament> though the really curious case is that this doesn't seem to be the case if the event is qualified @Default.
[17:09:15] <gastaldi> Strange
[17:10:20] <johnament> the funny thing I find is that the original weld docs were updated to the current 1.1.  the behavior is described correctly in 1.0.1 docs.
[17:10:51] <johnament> so when they updated the docs, no one bothered to check what it actually did :-)
[17:10:51] <gastaldi> But in the jcr impl we are not injecting event object with any qualifiers at all
[17:11:05] <gastaldi> Just selecting before firing
[17:11:28] <johnament> right, it has nothing to do with the event object, it's how weld chooses the observer methods.
[17:11:40] <gastaldi> Hum
[17:11:51] <johnament> they only check type.  but when you run in arquillian, it checks qualifiers as well.
[17:12:07] <johnament> arquillian in this case handles the observer
[17:12:11] <gastaldi> Hummm
[17:12:50] <gastaldi> Lets use arquillian as the cdi impl then :)
[17:16:38] <johnament> i'm also curious why that jackrabbit test case fails on hudson
[17:16:56] <gastaldi> File permissions maybe. ?
[17:17:20] <johnament> that's what i'm thinking, but you're only using build generated files right?
[17:17:30] <gastaldi> Yes
[17:17:36] <johnament> does jackrabbit have an in memory repository (not file system based)?
[17:17:46] <gastaldi> Not sure
[17:22:44] <gastaldi> Can you change that code to be just like it was in jcr impl ?
[17:23:07] <gastaldi> Just to be sure that its an Issue ?
[17:23:27] <gastaldi> In your ex app ?
[17:31:40] <gastaldi> ?
[17:35:16] <johnament> just finishing up.  same effect, almost
[17:35:21] <johnament> which makes no sense :-)
[17:36:07] <johnament> ok, take a look again.
[17:36:49] <johnament> when i visit this rest resource: http://localhost:8080/RestSending/send/s3/data no observes should get fired.
[17:37:08] <johnament> if you follow the parts of the specs that they say are wrong.
[17:38:11] <johnament> if you visit http://localhost:8080/RestSending/send/s1/data then only the observer that has qualone on it should get hit, but both qualone and the generic one get hit
[17:41:10] <gastaldi> It makes sense that the  generic should be hit also
[17:41:32] <johnament> what about the s3 example?
[17:41:41] <gastaldi> After all it assumes it does not Matter the qualifier you use
[17:42:32] 
[17:42:39] <johnament> no, all three
[17:43:05] <gastaldi> Hum. But you ate not selecting before firing
[17:43:47] <gastaldi> What about s2?
[17:44:23] <gastaldi> Which one is hit ?
[17:45:16] <johnament> qualtwo and default
[17:45:27] <johnament> take a look again, i created an s4 that selects instead of injecting.
[17:45:31] <johnament> same result, all three are hit
[17:45:32] <gastaldi> Correct
[17:45:46] <gastaldi> Let me see
[17:47:21] <gastaldi> Have you commited ?
[17:47:33] <johnament> yes.
[17:47:38] <johnament> i just added a 4th observer
[17:47:44] <johnament> all four get hit
[17:50:16] <gastaldi> That is correct also
[17:50:24] <johnament> how so?
[17:51:08] <gastaldi> When a qualifier is specified, it hits each observer
[17:51:24] <gastaldi> Including the default one
[17:51:30] <johnament> it hits any match.
[17:52:19] <johnament> so same type, or same qualifier, or if multiple qualifiers are specified then all of those qualifiers as well, as well as any combination of those qualifiers
[17:52:40] <gastaldi> I can see they are qualified
[17:53:02] <gastaldi> So when you select, it will be hit
[17:53:16] <gastaldi> Including the default one
[17:53:39] <gastaldi> Take S2 for example
[17:53:53] <johnament> what's the difference between selecting from an event and injecting the event with all of the qualifiers?
[17:54:30] <gastaldi> I dont think that injecting An. Event with qualifiers would work as you expect
[17:54:39] <gastaldi> But selecting will
[17:55:34] <gastaldi> Because you are qualifying the event producer
[17:55:42] <gastaldi> Not the consumer
[17:56:35] <johnament> gastaldi: the test case i put up there shows that there is no difference between the two
[17:58:05] <gastaldi> What test case you talking about ?
[17:58:15] <gastaldi> The app ?
[17:58:21] <johnament> yes
[17:59:13] <johnament> in both cases, s3 and s4 are observed by all four observer methods.
[18:00:15] <gastaldi> Because s3 is not selecting before firing ?
[18:01:15] <johnament> s3 is annotated
[18:01:18] <johnament> it doesn't need to select.
[18:01:45] <johnament> s4 uses select to find annotations
[18:02:09] <gastaldi> Its my understanding that the behavior is that if we select before firing it will hit any observes with the selected qualifier plus the default ones
[18:02:21] <gastaldi> Right ?
[18:02:46] <johnament> i'm not sure what that's based on
[18:03:08] <gastaldi> But behaves like this, ok ?
[18:03:30] <johnament> the inconsistency i'm pointing out is that in the spec, they say it should only hit observers with the same qualifiers.
[18:03:35] <johnament> then in a second part, it says all.
[18:03:44] <johnament> err any
[18:04:20] <gastaldi> I believe the second part is more alike
[18:04:31] <johnament> the second part is what was implemented.
[18:05:19] <johnament> my point (and why I removed the annotations) is that multiple observes would end up getting hit in the code base.
[18:05:56] <gastaldi> Not if they are qualified
[18:05:56] <gastaldi> Because they were selected
[18:06:18] <johnament> have you run the sample app? i can send you the output if you want.
[18:06:57] <johnament> i run this GET
[18:06:57] <johnament> http://localhost:8080/RestSending/send/s4/data
[18:07:04] <gastaldi> Let me guess
[18:07:09] <johnament> this is the log
[18:07:10] <johnament> 12:10:57,411 INFO  [com.tad.jms.restsending.SendEventResource] Observing the plain message.
[18:07:10] <johnament> 12:10:57,412 INFO  [com.tad.jms.restsending.SendEventResource] Observed QualOne
[18:07:10] <johnament> 12:10:57,412 INFO  [com.tad.jms.restsending.SendEventResource] Observed QualOne and QualTwo
[18:07:10] <johnament> 12:10:57,412 INFO  [com.tad.jms.restsending.SendEventResource] Observed QualTwo
[18:07:24] <johnament> s4 is the select style
[18:07:27] <gastaldi> Correct
[18:07:30] *** lincolnthree has quit IRC
[18:07:40] <johnament> this matches the second part.
[18:07:43] <gastaldi> All qualified plus the default
[18:07:53] <johnament> right
[18:07:57] <johnament> now here's s3
[18:08:06] *** lincolnthree has joined #seam-dev
[18:08:08] <johnament> http://localhost:8080/RestSending/send/s3/data
[18:08:16] <johnament> 12:12:18,494 INFO  [com.tad.jms.restsending.SendEventResource] Observing the plain message.
[18:08:16] <johnament> 12:12:18,495 INFO  [com.tad.jms.restsending.SendEventResource] Observed QualOne
[18:08:16] <johnament> 12:12:18,495 INFO  [com.tad.jms.restsending.SendEventResource] Observed QualOne and QualTwo
[18:08:16] <johnament> 12:12:18,496 INFO  [com.tad.jms.restsending.SendEventResource] Observed QualTwo
[18:08:20] <johnament> same results.
[18:09:07] <gastaldi> No selecting before firing. Correct also
[18:09:20] <johnament> it's qualified
[18:09:33] <johnament> qualified == selecting
[18:09:53] <gastaldi> Thats the point
[18:09:59] <gastaldi> As i said before
[18:10:50] <gastaldi> The current behavior should select before firing with the current qualifiers
[18:11:22] <gastaldi> But no qualifier were used on the jcr impl
[18:11:35] <johnament> so when the qualified events get fired, both @Observes @NodeAdded javax.jcr.obs..Event and @Observes javax.jcr.obs..Event are hit
[18:11:57] <gastaldi> Correct
[18:12:33] <gastaldi> Thats the expected behavior
[18:13:33] <gastaldi> Think of qualifiers as a filter
[18:14:02] <gastaldi> If no one is specified, then all will be hit
[18:14:16] <johnament> that is not correct.
[18:14:31] <gastaldi> You sure ?
[18:15:14] <gastaldi> Why not ?
[18:15:15] <johnament> yes, i'll send you a test case.
[18:16:03] <johnament> ok, look now.
[18:16:06] <johnament> when I hit s5
[18:16:12] <johnament> http://localhost:8080/RestSending/send/s5/data
[18:16:32] <johnament> this is the only result 12:19:36,304 INFO  [com.tad.jms.restsending.SendEventResource] Observing the plain message.
[18:17:22] <gastaldi> Correct
[18:17:37] <johnament> that's not what you just said though.
[18:17:38] <gastaldi> No surprises now
[18:18:20] <gastaldi> Thats what i meant though
[18:18:28] <gastaldi> Look
[18:18:33] <johnament> ok
[18:18:36] <johnament> no issue then.
[18:18:48] <johnament> i thought you had gotten it, but then i got confused in your last statement :-)
[18:19:00] <gastaldi> You are firing an event and not qualifying
[18:19:23] <gastaldi> Just the not qualified ones will be hit
[18:19:29] <gastaldi> Again, think as a filter
[18:21:21] <johnament> IMHO, the spec should be written as "An observer method will be notified of an event when all of the qualifiers on the field marked @Observes are found within the fired event."
[18:22:03] <gastaldi> What if you place @default ?
[18:22:15] <johnament> sae
[18:22:16] <johnament> same
[18:22:16] <gastaldi> In the not qualified one ?
[18:22:24] <johnament> no, won't get hit
[18:22:28] <johnament> same for @Any
[18:22:44] <gastaldi> Because you have a qualifier specified there
[18:22:54] <gastaldi> Maybe thats it
[18:22:55] <johnament> yes
[18:24:05] <johnament> so then if your expectation is that they're both hit, i'll put the annotation code back in last today
[18:24:28] <gastaldi> I would certainly expect that
[18:25:09] <gastaldi> Because they are always selected before firing
[18:25:41] <johnament> so what about the second half of SEAMJCR-8, letting the app developer determine when the listener gets bound?
[18:25:42] <jbossbot> jira [SEAMJCR-8] JCR Events mapped to CDI Events [Open (Unresolved) Feature Request, Major, George Gastaldi] https://issues.jboss.org/browse/SEAMJCR-8
[18:26:05] <gastaldi> Thats is a puzzle to solve yet
[18:26:26] <gastaldi> What about when creating a session ?
[18:27:24] <gastaldi> I meant, using the jcrlistener annotation
[18:28:01] <johnament> that does appear to work.
[18:28:09] <johnament> but i don't think the test case identifies it correctly.
[18:28:22] <johnament> since you manually register it.
[18:29:16] <gastaldi> The session must be logged in in order to call the listener
[18:29:47] <gastaldi> At least in jackrabbit
[18:30:37] <gastaldi> Did you find a way to pass the credentials to the session ?
[18:32:14] <johnament> we can make a way
[18:33:23] <gastaldi> I am still intrigued that we cannot inject a Repository or a Session without using that custom annotation
[18:33:50] <johnament> ModeShape has no concept of a default repository.
[18:33:58] <johnament> that jackrabbit seems to have.
[18:35:01] <gastaldi> Yes, you told that
[18:36:27] <gastaldi> Gotta go now
[18:36:31] <gastaldi> Check u later
[18:36:57] *** gastaldi has left #seam-dev
[18:37:35] *** johnament has quit IRC
[18:59:59] *** lincolnthree has quit IRC
[19:27:31] *** lincolnthree has joined #seam-dev
[19:29:43] *** maxandersen has joined #seam-dev
[19:30:44] <jbossbot> git [validation] push master dd88403.. Gunnar Morling Merge branch 'SEAMVALIDATE-4' of https://github.com/mgencur/validation into mgencur-SEAMVALIDATE-4
[19:30:45] <jbossbot> jira [SEAMVALIDATE-4] Helloworld example throwing "java.lang.LinkageError: loader constraint violation" when deployed into JBossAS 6 [Reopened (Unresolved) Bug, Major, Unassigned] https://issues.jboss.org/browse/SEAMVALIDATE-4
[19:30:46] <jbossbot> git [validation] push master 31cc306.. Gunnar Morling SEAMVALIDATE-4: Enabling example to run on GlassFish out-of-the-box
[19:30:46] <jbossbot> git [validation] push master URL: http://github.com/seam/validation/compare/6f622ea...31cc306
[19:51:45] *** lincolnthree has quit IRC
[20:22:10] *** mgencur has joined #seam-dev
[20:22:20] *** mgencur has left #seam-dev
[21:02:50] *** gastaldi has joined #seam-dev
[21:03:20] *** gastaldi has quit IRC
[21:44:21] <jbossbot> git [validation] push master 5b9b804.. Gunnar Morling SEAMVALIDATE-8: Simplifying configuration of Weld/CDI versions in POMs
[21:44:22] <jbossbot> jira [SEAMVALIDATE-8] Align context root for Jetty tests with module name [Coding In Progress (Unresolved) Enhancement, Major, Gunnar Morling] https://issues.jboss.org/browse/SEAMVALIDATE-8
[21:44:22] <jbossbot> git [validation] push master 5f1c84f.. Gunnar Morling SEAMVALIDATE-8: Using same context root for Jetty based tests as for JBoss/GF
[21:44:22] <jbossbot> git [validation] push master URL: http://github.com/seam/validation/compare/31cc306...5f1c84f
[21:45:12] *** maxandersen has quit IRC
[21:50:03] *** lincolnthree has joined #seam-dev
[21:52:30] *** lincolnthree has quit IRC
[21:52:34] *** gastaldi has joined #seam-dev
[21:52:45] *** gastaldi has left #seam-dev
[22:01:34] *** gastaldi has joined #seam-dev
[22:01:47] *** gastaldi has left #seam-dev
[22:03:44] *** jose_f_home has joined #seam-dev
[22:04:26] *** jose_freitas has quit IRC
[22:32:21] *** bitshuffler_ has quit IRC
[22:54:06] *** nickarls has quit IRC
[23:02:32] *** gastaldi has joined #seam-dev
[23:02:52] *** gastaldi has left #seam-dev
[23:18:08] *** johnament has joined #seam-dev
[23:52:21] <jbossbot> git [jcr] push master cd94de4.. John Ament SEAMJCR-8 Readded annotation support to fired events after confirming behavior.
[23:52:23] <jbossbot> jira [SEAMJCR-8] JCR Events mapped to CDI Events [Open (Unresolved) Feature Request, Major, George Gastaldi] https://issues.jboss.org/browse/SEAMJCR-8
[23:52:23] <jbossbot> git [jcr] push master URL: http://github.com/seam/jcr/compare/6a95fdc...cd94de4
[23:53:16] *** sbryzak has joined #seam-dev
[23:53:17] *** sbryzak has quit IRC
[23:53:17] *** sbryzak has joined #seam-dev





top