Hi Ian, Those are newly added collections.
On Tue, Sep 10, 2013 at 12:00 PM, Ian Boston <i...@tfd.co.uk> wrote: > 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 > > > -- Thanks /Dishara
