Re: enhancements/fixes for slide 2.1

Eirikur Hrafnsson <[email protected]>
Newsgroups gmane.comp.jakarta.slide.devel
Message-ID <[email protected]>
Hi Max and Slide list...

I was wondering if your fixes to the DASL ordering and limiting have  
been added to Slide yet?
I'm working with a 6 months old Slide build or so and ordering seems  
to be broken although limiting works.

Ordering by getlastmodified e.g. in this DASL search does not order  
the results:

<D:searchrequest xmlns:D='DAV:' xmlns:S='http://jakarta.apache.org/ 
slide/'>
<D:basicsearch>
<D:select><D:prop><D:getcontentlength/><D:creationdate/ 
 ><D:displayname/><D:getlastmodified/></D:prop></D:select>
<D:from><D:scope><D:href>files</D:href><D:depth>infinity</D:depth></ 
D:scope></D:from>
<D:where>
<D:and>
<D:or>
<D:contains>***</D:contains>
<S:property-contains><D:prop><D:displayname/></D:prop><D:literal>***</ 
D:literal>
</S:property-contains><S:property-contains>
<D:prop><D:creator-displayname/></D:prop><D:literal>***</D:literal>
</S:property-contains>
<S:property-contains><D:prop><D:comment/></D:prop><D:literal>***</ 
D:literal></S:property-contains>
</D:or>
<D:not-is-collection/>
</D:and>
</D:where>
<D:orderby><D:order><D:prop><D:getlastmodified/></ 
D:prop><D:descending/></D:order></D:orderby>
</D:basicsearch></D:searchrequest>



On 20.2.2006, at 11:09, Max Pfingsthorn wrote:

> Dear all,
>
> I hate to reply to myself, but I was just wondering if the offer  
> still stands... I would love to help out and it would be great if  
> you guys gave me that chance!
>
> Best,
> Max
>
>> -----Original Message-----
>> From: Max Pfingsthorn
>> Sent: Tuesday, February 14, 2006 11:20
>> To: Slide Developers Mailing List
>> Subject: RE: enhancements/fixes for slide 2.1
>>
>>
>> Hello everyone!
>>
>>> -----Original Message-----
>>> From: [email protected] [mailto:[email protected]]
>>> Sent: Saturday, February 11, 2006 12:33
>>> To: 'Slide Developers Mailing List'
>>> Subject: AW: enhancements/fixes for slide 2.1
>>>
>>> Hi Hippo!
>>
>> Hippo is just my employer ;) Just call me Max.
>>
>>> Fromm y point of view it would be great if you would become a slide
>>> committer and could add patches to the sources directly. If you are
>>> interested, I would start a voting thread on this mailing list.
>>
>> That would be great! I would be very honored.
>>
>>> I think most of your patches are very helpful for the slide
>> community.
>>
>> Thanks, I do my best.
>>
>>> One comment to your patches regarding the lucene-based search:
>>> If you use lucene directly to limit the results this might
>> cause some
>>> trouble if you are using acl's. The acl's get evaluated after slide
>>> retrieves the search result from lucene. If you for example
>>> specify a limit
>>> of 10 results and the user executing the DASL-request has no
>>> permissions to
>>> read half of them, he would only get 5 files even if he
>>> specified to get 10.
>>> So it would be very helpful to have a configurable switch to
>>> select if you
>>> want to have a fast system (with the risk of wrong number of
>>> results) or a
>>> slower search with correct results.
>>
>> Actually, you cannot tell Lucene to only return so many hits.
>> The nice thing about Lucene's Hits datastructure is that it
>> lazy-loads data from the index. That way, it can return the
>> full structure, without much of the overhead, and you can
>> decide how many you access yourself. What I do is to iterate
>> through the Hits until I generated as many authorized results
>> as needed. What makes it fast is that sorting is done in
>> Lucene as well, so even then, you don't have to load all
>> results in memory and sort them there as it's done right now.
>>
>>> Why have you used JMS to notify cocoon? Have you had a look at the
>>> Notification abilities in the slide webdavlib?
>>
>> I wasn't aware that the webdavlib had such a feature. Also,
>> when I joined the team here, JMS notification was already  
>> implemented.
>>
>> Bye!
>> max
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: [email protected]
>> For additional commands, e-mail: [email protected]
>>
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
>
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.