[infinispan-dev] Fwd: [JBoss JIRA] (ISPN-1629) REST server test suite should use 5.1 namespace
Pete, since the introduction of the Stax parser, the REST test suite was failing because it could not parse the 5.0 namespace. I solved it by changing it, but I was wondering whether we should also be able to parse 5.0 configuration files, since we're in the same major version ? Tristan Original Message Subject: [JBoss JIRA] (ISPN-1629) REST server test suite should use 5.1 namespace Date: Tue, 20 Dec 2011 03:21:09 -0500 (EST) From: Tristan Tarrant (Created) (JIRA) jira-eve...@lists.jboss.org To: ttarr...@redhat.com REST server test suite should use 5.1 namespace --- Key: ISPN-1629 URL: https://issues.jboss.org/browse/ISPN-1629 Project: Infinispan Issue Type: Bug Reporter: Tristan Tarrant Assignee: Tristan Tarrant Since the introduction of the new Stax parser, the REST tests are failing because the config xml uses the 5.0 namespace. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev
Re: [infinispan-dev] [JBoss JIRA] (ISPN-1629) REST server test suite should use 5.1 namespace
+1 as well. On 20 Dec 2011, at 11:27, Sanne Grinovero wrote: +1 backwards compatibility seems important to me, especially in the same mayor version. Sanne On 20 December 2011 12:24, Pete Muir pm...@redhat.com wrote: We certainly can do. It's not difficult to fix. By default I allowed us to parse 5.1 and un-namespaced files. Thoughts? I think 5.0 and 5.1 are 100% compatible, so it's just a 1 line change. On 20 Dec 2011, at 08:36, Tristan Tarrant wrote: Pete, since the introduction of the Stax parser, the REST test suite was failing because it could not parse the 5.0 namespace. I solved it by changing it, but I was wondering whether we should also be able to parse 5.0 configuration files, since we're in the same major version ? Tristan Original Message Subject: [JBoss JIRA] (ISPN-1629) REST server test suite should use 5.1 namespace Date: Tue, 20 Dec 2011 03:21:09 -0500 (EST) From: Tristan Tarrant (Created) (JIRA) jira-eve...@lists.jboss.org To: ttarr...@redhat.com REST server test suite should use 5.1 namespace --- Key: ISPN-1629 URL: https://issues.jboss.org/browse/ISPN-1629 Project: Infinispan Issue Type: Bug Reporter: Tristan Tarrant Assignee: Tristan Tarrant Since the introduction of the new Stax parser, the REST tests are failing because the config xml uses the 5.0 namespace. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev -- Manik Surtani ma...@jboss.org twitter.com/maniksurtani Lead, Infinispan http://www.infinispan.org ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev
Re: [infinispan-dev] Distributed Executors in write mode, need reference to Cache
On 20 December 2011 13:35, Manik Surtani ma...@jboss.org wrote: What else do you think could/should be provided? Whatever people want to do with their data. Imagine you have to update all values by using some external service which provides some needed input, or you want to find all entries by some criteria and send them over to S3 via another service, or are storing movies and want to encode them using some other tool you have - these services could be invoked each time but assuming it has some expensive initialization it's likely one wants to inject references to initialized services and pools of resources. This user is asking for a reference for the Cache, that's the first step, but the next step is obviously to inject anything out of our control; recommending a new JNDI context + lookup per entry is likely not the most usable approach, while CDI should be simple to integrate and provide lot of value. On 19 Dec 2011, at 18:57, Sanne Grinovero wrote: Highlighting this use case: http://community.jboss.org/message/642300#642300 As I replied on the forum, I think we might need to improve the DistributedExecutorService by providing access to some context to the Future before it's executed remotely, as having a reference to the Cache it's being executed on seems very useful, but it might not be limited to the Cache only. I guess to make good use of it, someone might need to access just anything: CDI looks like a good fit? ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev -- Manik Surtani ma...@jboss.org twitter.com/maniksurtani Lead, Infinispan http://www.infinispan.org ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev
Re: [infinispan-dev] Distributed Executors in write mode, need reference to Cache
So you're suggesting that the task is registered as a CDI bean on each remote node, so that requested services can be injected? Makes sense. But this would have to be a part of infinispan-cdi, as an extension to what we have in DistExec. I presume this is what you had in mind? On 20 Dec 2011, at 12:43, Sanne Grinovero wrote: On 20 December 2011 13:35, Manik Surtani ma...@jboss.org wrote: What else do you think could/should be provided? Whatever people want to do with their data. Imagine you have to update all values by using some external service which provides some needed input, or you want to find all entries by some criteria and send them over to S3 via another service, or are storing movies and want to encode them using some other tool you have - these services could be invoked each time but assuming it has some expensive initialization it's likely one wants to inject references to initialized services and pools of resources. This user is asking for a reference for the Cache, that's the first step, but the next step is obviously to inject anything out of our control; recommending a new JNDI context + lookup per entry is likely not the most usable approach, while CDI should be simple to integrate and provide lot of value. On 19 Dec 2011, at 18:57, Sanne Grinovero wrote: Highlighting this use case: http://community.jboss.org/message/642300#642300 As I replied on the forum, I think we might need to improve the DistributedExecutorService by providing access to some context to the Future before it's executed remotely, as having a reference to the Cache it's being executed on seems very useful, but it might not be limited to the Cache only. I guess to make good use of it, someone might need to access just anything: CDI looks like a good fit? ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev -- Manik Surtani ma...@jboss.org twitter.com/maniksurtani Lead, Infinispan http://www.infinispan.org ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev -- Manik Surtani ma...@jboss.org twitter.com/maniksurtani Lead, Infinispan http://www.infinispan.org ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev
Re: [infinispan-dev] Distributed Executors in write mode, need reference to Cache
On 20 December 2011 13:47, Manik Surtani ma...@jboss.org wrote: So you're suggesting that the task is registered as a CDI bean on each remote node, so that requested services can be injected? Makes sense. But this would have to be a part of infinispan-cdi, as an extension to what we have in DistExec. I presume this is what you had in mind? Yes. Anyway a first step would be to provide a reference to the Cache, both to DistExec and MapReduce tasks, this should be doable even without introducing infinispan-cdi; but we need a cool proposal in terms of API - especially if @Inject is not supported that makes it harder to design. Sanne On 20 Dec 2011, at 12:43, Sanne Grinovero wrote: On 20 December 2011 13:35, Manik Surtani ma...@jboss.org wrote: What else do you think could/should be provided? Whatever people want to do with their data. Imagine you have to update all values by using some external service which provides some needed input, or you want to find all entries by some criteria and send them over to S3 via another service, or are storing movies and want to encode them using some other tool you have - these services could be invoked each time but assuming it has some expensive initialization it's likely one wants to inject references to initialized services and pools of resources. This user is asking for a reference for the Cache, that's the first step, but the next step is obviously to inject anything out of our control; recommending a new JNDI context + lookup per entry is likely not the most usable approach, while CDI should be simple to integrate and provide lot of value. On 19 Dec 2011, at 18:57, Sanne Grinovero wrote: Highlighting this use case: http://community.jboss.org/message/642300#642300 As I replied on the forum, I think we might need to improve the DistributedExecutorService by providing access to some context to the Future before it's executed remotely, as having a reference to the Cache it's being executed on seems very useful, but it might not be limited to the Cache only. I guess to make good use of it, someone might need to access just anything: CDI looks like a good fit? ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev -- Manik Surtani ma...@jboss.org twitter.com/maniksurtani Lead, Infinispan http://www.infinispan.org ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev -- Manik Surtani ma...@jboss.org twitter.com/maniksurtani Lead, Infinispan http://www.infinispan.org ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev
Re: [infinispan-dev] Some flags are incompatible with implicit transactions
On 14 Dec 2011, at 13:55, Galder Zamarreño wrote: Yeah it's pointless to use locks if you have a single operation, but I might want to do more operations in a single transaction.. actually what's the point of using a transaction if I have only one operation? WIthout transaction it is possible that the operation is only partially applied, i.e. on a subset of numOwners, resulting in inconsistent state. Good point Mircea. You should definitely document that cos that's pretty much the sole reason to use implicit transactions, something users should know. Yes, please make sure this makes it onto Confluence. -- Manik Surtani ma...@jboss.org twitter.com/maniksurtani Lead, Infinispan http://www.infinispan.org ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev
Re: [infinispan-dev] Distributed Executors in write mode, need reference to Cache
On 20 December 2011 14:07, Manik Surtani ma...@jboss.org wrote: Yup, I agree. Care to create JIRAs for these two? I think the first one (adding a cache reference to the task) should be for 5.1.0. For 5.1: https://issues.jboss.org/browse/ISPN-1634 For 6.0 (?) : https://issues.jboss.org/browse/ISPN-1636 ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev
Re: [infinispan-dev] Distributed Executors in write mode, need reference to Cache
On 11-12-20 9:54 AM, Sanne Grinovero wrote: On 20 December 2011 13:47, Manik Surtanima...@jboss.org wrote: So you're suggesting that the task is registered as a CDI bean on each remote node, so that requested services can be injected? Makes sense. But this would have to be a part of infinispan-cdi, as an extension to what we have in DistExec. I presume this is what you had in mind? Yes. Anyway a first step would be to provide a reference to the Cache, both to DistExec and MapReduce tasks, this should be doable even without introducing infinispan-cdi; but we need a cool proposal in terms of API - especially if @Inject is not supported that makes it harder to design. Sanne Makes sense! Note that implementors of DistributedCallable task in distributed executors already receive Cache instance through API! We can have Mapper and Reducer receive owning cache instance through API as well! However, if it can be done through @Inject mechanism maybe that would be better! ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev
Re: [infinispan-dev] Distributed Executors in write mode, need reference to Cache
Vladimir, for you? :) On 20 Dec 2011, at 13:21, Sanne Grinovero wrote: On 20 December 2011 14:07, Manik Surtani ma...@jboss.org wrote: Yup, I agree. Care to create JIRAs for these two? I think the first one (adding a cache reference to the task) should be for 5.1.0. For 5.1: https://issues.jboss.org/browse/ISPN-1634 For 6.0 (?) : https://issues.jboss.org/browse/ISPN-1636 ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev -- Manik Surtani ma...@jboss.org twitter.com/maniksurtani Lead, Infinispan http://www.infinispan.org ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev
Re: [infinispan-dev] Distributed Executors in write mode, need reference to Cache
Sure, I'd like to hear Pete's opinion here. Can we inject pretty much anything from application scope or do we have to do something special in Mapper and Reducer to prepare objects that can be injected? I'd rather leave interfaces as they are and use injection mechanism to inject objects needed by implementors of Mapper and Reducer. Today it is cache, tomorrow people will ask for something else...and it seems to me injection mechanism is the way to go here! Pete? Vladimir On 11-12-20 10:55 AM, Manik Surtani wrote: Vladimir, for you? :) On 20 Dec 2011, at 13:21, Sanne Grinovero wrote: On 20 December 2011 14:07, Manik Surtanima...@jboss.org wrote: Yup, I agree. Care to create JIRAs for these two? I think the first one (adding a cache reference to the task) should be for 5.1.0. ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev
Re: [infinispan-dev] Distributed Executors in write mode, need reference to Cache
Are you asking what else needs to be added to the Mapper and Reducer interfaces to make them CDI-friendly so that implementations can have stuff injected? I suspect nothing need be done except that on the receiving node, if these implementations are annotated as CDI beans, then they should be passed to the CDI container for proper setup before being used. On 20 Dec 2011, at 14:00, Vladimir Blagojevic wrote: Sure, I'd like to hear Pete's opinion here. Can we inject pretty much anything from application scope or do we have to do something special in Mapper and Reducer to prepare objects that can be injected? I'd rather leave interfaces as they are and use injection mechanism to inject objects needed by implementors of Mapper and Reducer. Today it is cache, tomorrow people will ask for something else...and it seems to me injection mechanism is the way to go here! Pete? Vladimir On 11-12-20 10:55 AM, Manik Surtani wrote: Vladimir, for you? :) On 20 Dec 2011, at 13:21, Sanne Grinovero wrote: On 20 December 2011 14:07, Manik Surtanima...@jboss.org wrote: Yup, I agree. Care to create JIRAs for these two? I think the first one (adding a cache reference to the task) should be for 5.1.0. ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev -- Manik Surtani ma...@jboss.org twitter.com/maniksurtani Lead, Infinispan http://www.infinispan.org ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev
Re: [infinispan-dev] Distributed Executors in write mode, need reference to Cache
Yeah, that is my question. In conclusion, Mapper/Reducer instances should be passed to CDI container for resolution once they are migrated to execution nodes, but before they are executed on those nodes On 11-12-20 12:29 PM, Manik Surtani wrote: Are you asking what else needs to be added to the Mapper and Reducer interfaces to make them CDI-friendly so that implementations can have stuff injected? I suspect nothing need be done except that on the receiving node, if these implementations are annotated as CDI beans, then they should be passed to the CDI container for proper setup before being used. On 20 Dec 2011, at 14:00, Vladimir Blagojevic wrote: Sure, I'd like to hear Pete's opinion here. Can we inject pretty much anything from application scope or do we have to do something special in Mapper and Reducer to prepare objects that can be injected? I'd rather leave interfaces as they are and use injection mechanism to inject objects needed by implementors of Mapper and Reducer. Today it is cache, tomorrow people will ask for something else...and it seems to me injection mechanism is the way to go here! Pete? Vladimir On 11-12-20 10:55 AM, Manik Surtani wrote: Vladimir, for you? :) On 20 Dec 2011, at 13:21, Sanne Grinovero wrote: On 20 December 2011 14:07, Manik Surtanima...@jboss.org wrote: Yup, I agree. Care to create JIRAs for these two? I think the first one (adding a cache reference to the task) should be for 5.1.0. ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev -- Manik Surtani ma...@jboss.org twitter.com/maniksurtani Lead, Infinispan http://www.infinispan.org ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev
Re: [infinispan-dev] Distributed Executors in write mode, need reference to Cache
That should be pretty trivial. Only hard bit is integrating this into CDI module. Vladimir, do you provide a SPI for object instantiation and injection in the Map Reduce/Distributed Executor stuff? If so, Kevin can probably knock it up in an hour or two + tests ;-) On 20 Dec 2011, at 16:47, Vladimir Blagojevic wrote: Yeah, that is my question. In conclusion, Mapper/Reducer instances should be passed to CDI container for resolution once they are migrated to execution nodes, but before they are executed on those nodes On 11-12-20 12:29 PM, Manik Surtani wrote: Are you asking what else needs to be added to the Mapper and Reducer interfaces to make them CDI-friendly so that implementations can have stuff injected? I suspect nothing need be done except that on the receiving node, if these implementations are annotated as CDI beans, then they should be passed to the CDI container for proper setup before being used. On 20 Dec 2011, at 14:00, Vladimir Blagojevic wrote: Sure, I'd like to hear Pete's opinion here. Can we inject pretty much anything from application scope or do we have to do something special in Mapper and Reducer to prepare objects that can be injected? I'd rather leave interfaces as they are and use injection mechanism to inject objects needed by implementors of Mapper and Reducer. Today it is cache, tomorrow people will ask for something else...and it seems to me injection mechanism is the way to go here! Pete? Vladimir On 11-12-20 10:55 AM, Manik Surtani wrote: Vladimir, for you? :) On 20 Dec 2011, at 13:21, Sanne Grinovero wrote: On 20 December 2011 14:07, Manik Surtanima...@jboss.org wrote: Yup, I agree. Care to create JIRAs for these two? I think the first one (adding a cache reference to the task) should be for 5.1.0. ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev -- Manik Surtani ma...@jboss.org twitter.com/maniksurtani Lead, Infinispan http://www.infinispan.org ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev
[infinispan-dev] inefficient usage of CacheLoader with REPL
Hi all, I just noticed writing a test using REPL that when doing a put operation the other nodes do a CacheLoader LOAD as well. Isn't that totally unnecessary? In fact assuming REPL guarantees consistency across all nodes, any form of remote return value should be implicitly skipped as we can satisfy return values by looking at local copies. Sanne ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev