Hi Dishara, Great, thanks. What do the LA, MA, and SA stand for ? Best regards Ian
On Tuesday, September 10, 2013, Dishara Wijewardana wrote: > Hi Ian > Also commited the 3 new test classes for CUD to google code repo. > > > On Tue, Sep 10, 2013 at 10:30 AM, Dishara Wijewardana < > ddwijeward...@gmail.com <javascript:;>> wrote: > > > Hi Ian, > > Finally was able to fix ;-). Had to debug along with the path and fix > some > > other issues also in Cassandra Provider impl. I have updated the JIRA > with > > the new reports of CUD. > > (I ran them one after the other respectively) > > > > In brief as follows. > > CREATE > > Average Latency Under Node LA = 165 (ms) > > Average Latency Under Node MA = 203 (ms) > > Average Latency Under Node SA = 256 (ms) > > #TOTAL CALLS = 300 Total Average Latency = 208 (ms) > > > > UPDATE > > Average Latency Under Node LA = 36 (ms) > > Average Latency Under Node MA = 29 (ms) > > Average Latency Under Node SA = 25 (ms) > > #TOTAL CALLS = 300 Total Average Latency = 30 (ms) > > > > DELETE > > Average Latency Under Node LA = 20 (ms) > > Average Latency Under Node MA = 19 (ms) > > Average Latency Under Node SA = 19 (ms) > > #TOTAL CALLS = 300 Total Average Latency = 19 (ms) > > > > > > > > > > > > > > > > On Mon, Sep 9, 2013 at 5:47 PM, Ian Boston <i...@tfd.co.uk> wrote: > > > >> Hi > >> The CassandraImpl needs to return something suitable to represent > >> > >> /content/cassandra/p > >> > >> For the moment, if the column family p does exist, return a new type > >> of read only resource, eg CassandraColumnFamilyResource. > >> > >> Ian > >> > >> > >> On 9 September 2013 13:02, Dishara Wijewardana <ddwijeward...@gmail.com > > > >> wrote: > >> > On Mon, Sep 9, 2013 at 2:09 PM, Ian Boston <i...@tfd.co.uk> wrote: > >> > > >> >> Hi, > >> >> The call to processCreate a few lines earlier should have created the > >> >> resource so that the subsequent call to getResource returned the new > >> >> resource into newResource. > >> >> > >> >> If you look through the calls the create operation is called in > >> >> > >> >> > >> >> > >> > org.apache.sling.servlets.post.impl.operations.AbstractCreateOperation.deepGetOrCreateNode(ResourceResolver, > >> >> String, Map<String, RequestProperty>, List<Modification>, > >> >> VersioningConfiguration) > >> >> > >> >> Line 533 > >> >> > >> >> resource = resolver.create(resource, name, > props); > >> >> > >> >> Stick a breakpoint at that line and see, if its called, and if the > >> >> resource resolver calls your code. > >> >> > >> >> > >> >> > >> > Hi Ian, > >> > Yes it calls the create method. As I see the issue is the commit not > >> > getting called and hence resolver cannot get the created resource. > >> > So I thought of commit on the fly @create method to verify that. But > we > >> > have a problem here. > >> > > >> > When I try to create /content/cassandra/p/c node, as you exactly > >> mentioned > >> > before, it tries to create /content/cassandra/p node. In Cassandra > >> Impl, > >> > there cannot be a node like /content/cassandra/p and hence returns > >> NULL as > >> > earlier and still create fails. Because "p" is the column family (the > >> model > >> > which we agreed and implemented). So what is the best approach to > >> overcome > >> > this. > >> > > >> > > >> > > >> >> BTW: Its Ok that there is a newResource.adaptTo(Node.class) as the > >> >> result is checked for Null a moment later. You do not need to support > >> >> adaptTo(Node.class), which is a special case for Jcr. > >> >> > >> >> Best Regards > >> >> Ian > >> >> > >> >> On 9 September 2013 06:05, Dishara Wijewardana < > >> ddwijeward...@gmail.com> > >> >> wrote: > >> >> > Hi Ian, > >> >> > I debug the servlet from line to line and locate the NPE that > causes > >> this > >-- > Thanks > /Dishara >
