Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 339193 - RecordStorageException while crawling: error storing record id: null
Summary: RecordStorageException while crawling: error storing record id: null
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Smila (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Daniel Stucky CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-03-08 05:34 EST by Peter Wissel CLA
Modified: 2022-07-07 11:31 EDT (History)
0 users

See Also:

Attachments
log file with stacktrace (7.10 KB, text/plain)
2011-03-08 05:35 EST, Peter Wissel CLA 		no flags Details
possible solution patch (855 bytes, patch)
2011-03-08 05:40 EST, Peter Wissel CLA 		igor.novakovic: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Peter Wissel CLA 2011-03-08 05:34:01 EST 
Build Identifier: M20100909-0800

I crawl a folder several times. If a file was deleted between two crawl jobs an RecordStorageException occurs (see attachment).

Reproducible: Always

Steps to Reproduce:
1. Crawl a folder.
2. Delete a file in this folder.
3. Crawl the folder again.
Comment 1 Peter Wissel CLA 2011-03-08 05:35:39 EST 
Created attachment 190641 [details]
log file with stacktrace
Comment 2 Peter Wissel CLA 2011-03-08 05:40:17 EST 
Created attachment 190642 [details]
possible solution patch

After research I think the problem comes with revision 604.
A record is created in deleteDelta method via RecordFactory.DEFAULT_INSTANCE but no Id is set.
Comment 3 Igor Novakovic CLA 2011-03-11 04:23:13 EST 
Daniel, could you please take a quick look at this?

Regards
Igor
Comment 4 Daniel Stucky CLA 2011-03-14 07:06:58 EDT 
Hi Peter,

thanks for providing this patch. I just tested it and it fixes the problem.
I checked in your patch as revision #966
Comment 5 Andreas Weber CLA 2013-04-15 11:49:38 EDT 
Closing this