After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 315056 - Child indexables on the filesystem
Child indexables on the filesystem
Status: RESOLVED FIXED
Product: beagle
Classification: Other
Component: General
unspecified
Other All
: Normal normal
: ---
Assigned To: Debajyoti Bera
Jon Trowbridge
Depends on:
Blocks:
 
 
Reported: 2005-09-01 18:49 UTC by Veerapuram Varadhan
Modified: 2006-11-24 21:54 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Propose fix and FilterArchive (14.44 KB, patch)
2005-09-01 18:54 UTC, Veerapuram Varadhan
none Details | Review
Indexable fix (2.50 KB, patch)
2005-10-21 13:38 UTC, Veerapuram Varadhan
none Details | Review
Enable child indexables on the filesystem (3.00 KB, patch)
2005-10-22 10:02 UTC, Daniel Drake
none Details | Review

Description Veerapuram Varadhan 2005-09-01 18:49:52 UTC
Please describe the problem:
The attached patch contains a beagle filter for archive files.  Archives are
opened and the individual entries are treated as Child indexables.  When
indexing child indexables, they will have a preset BinaryStream or TextReader. 
So, the GetTextReader () or GetBinaryStream () should check for the existance of
a textreader or binary_stream and in the absence of both, should try to create
one from the contenturi.

Steps to reproduce:
1. 
2. 
3. 


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Veerapuram Varadhan 2005-09-01 18:54:39 UTC
Created attachment 51672 [details] [review]
Propose fix and FilterArchive

Patch contains a proposed fix for the problem mentioned in the bug description
and FilterArchive that triggered the bug. ;-)
Comment 2 Veerapuram Varadhan 2005-10-21 13:38:37 UTC
Created attachment 53729 [details] [review]
Indexable fix

Splitting up the patch in to :
1) Archive filter
2) Indexable fix.

Attaching the indexable fix here and creating a new bug for filter archive to
attach the "archive filter"
Comment 3 Daniel Drake 2005-10-22 10:02:39 UTC
Created attachment 53756 [details] [review]
Enable child indexables on the filesystem

This enables child indexables on the filesystem.

I'm not too sure about the changes to RemapUri (adding "parent:" to property
keys).

Say we had a maildir file with attachment inside a zip file, the email
attachment would have a Uri such as:

archive.zip#email#0

The lucene document could either have parent:beagle:ExactFilename of
"archive.zip" or "email". Depends whether we want this to be the real
'physical' parent, or the actual parent. My hack here assumes it would be
archive.zip (but I haven't tested any doubly-nested files yet).

Alternatively we could always ensure beagle:ExactFilename points to the
physical parent file?
Comment 4 Joe Shaw 2006-01-23 19:24:10 UTC
Jon, can you fill us in on the current state of things?
Comment 5 Jon Trowbridge 2006-01-23 21:09:08 UTC
The patch is forthcoming, but my schedule got extremely covoluted --- among other things, my start date at Google got moved up a week.  (I'm sitting in new employee orientation even as a type...)
Comment 6 Kevin Kubasik 2006-09-16 05:30:00 UTC
These patches have archival value, but no longer apply or compile, check out the FilterArchive bug for a patch that includes these and more.
Comment 7 Debajyoti Bera 2006-11-24 21:54:27 UTC
Initial implementation in CVS, based on patches here and in the archive-filter bug. It handles nested child indexables, marking indexables done only after all children are done, remapping child uri to return the correct uri fullpath#child1#child2#nested.document to clients. Requesting testing. Marking RESOLVED.