Bug 65108 - CSV Importing Problems - Slowness
Summary: CSV Importing Problems - Slowness
Status: RESOLVED UNMAINTAINED
Alias: None
Product: kab3
Classification: Miscellaneous
Component: general (show other bugs)
Version: 3.1
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Tobias Koenig
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-09-28 19:05 UTC by Ben Lamb
Modified: 2009-08-05 15:59 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
CSV Formatted list of KDE CVS account holders (34.37 KB, text/plain)
2003-09-28 20:18 UTC, Ben Lamb
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ben Lamb 2003-09-28 19:05:18 UTC
Version:           3.1 (using KDE 3.1.92 (alpha2, CVS >= 20030921), compiled sources)
Compiler:          gcc version 3.2.2
OS:          Linux (i686) release 2.4.21

I tried to import a 400 row CSV file. It was exported from a previous version KAddressbook so has all the columns. File size is 91Kb.

I identified about five columns and started the import. Dialog disappeared but the main window didn't redraw, CPU load was very high and the application remained in that state for five minutes until I killed it.

Some of the contacts, maybe half, seem to have been imported.

Please, is it possible to either speed it up or show a progress dialog. If you don't have time to implement these perhaps a warning dialog that the function is slow and may take several minutes? Otherwise impatient people like me will think it just doesn't work :-).

One other thing. My dataset included rows with multiple lines e.g.

"A Very Large Company
Widgets Division"

These are imported correctly but sometimes the field they were imported into doesn't allow multiple lines, e.g. Organisation. This causes a very minor display problem and was easily fixed by manually editing the record. Ideally the carriage returns would be stripped out for fields that don't support multiple lines.

Thanks,

Ben.
Comment 1 Tobias Koenig 2003-09-28 19:43:13 UTC
Subject: Re:  New: CSV Importing Problems - Slowness

On Sun, Sep 28, 2003 at 05:05:19PM -0000, Ben Lamb wrote:
> Version:           3.1 (using KDE 3.1.92 (alpha2, CVS >= 20030921), compiled sources)
> Compiler:          gcc version 3.2.2
> OS:          Linux (i686) release 2.4.21
Hi Ben,

> I tried to import a 400 row CSV file. It was exported from a
> previous version KAddressbook so has all the columns. File size is 91Kb.
Could you send me the file via private mail for performance
testing/debugging?

Ciao,
Tobias
Comment 2 Ben Lamb 2003-09-28 20:18:01 UTC
Created attachment 2614 [details]
CSV Formatted list of KDE CVS account holders

Here's a simpler file that exhibits the same problem. It's the KDE CVS account
list in CSV format. I idenitifed the columns as Role, Given Name, Email
Address.
Comment 3 Ben Lamb 2003-09-28 20:29:39 UTC
It got as far as katz, line 374, before I killed it. I then selecting all those users and 
deleting them en-mass. It's taking an age with no progress indicator or redrawing it 
looks like the program has crashed. 
 
Okay, this isn't something many people are likely to do every day. It would good to use 
this file to do some performance testing. Having addressbooks with several hundred 
entries in them isn't unusual. I've got a fairly fast machine and it's not doing anything 
processor intensive at the moment. 
 
I hate to moan but KAddressbook, in fact the whole of KDE Pim, is looking really great 
and I want to be able to do day-to-day work with it as it has better functionality than 
Palm Desktop which I currently use. 
 
Thanks, 
 
Ben. 
Comment 4 Tobias Koenig 2003-09-29 15:32:53 UTC
Subject: kdepim/kaddressbook

CVS commit by tokoe: 

More intelligent locking => better performance.
CCMAIL:65108-done@bugs.kde.org


  M +1 -1      kablock.cpp   1.3
  M +35 -11    undocmds.cpp   1.18
  M +5 -0      xxportmanager.cpp   1.15



Comment 5 Ben Lamb 2003-09-30 10:12:19 UTC
Thank-you very much Tobias. This is fantastic, > 4 min to about 10 seconds, with a 
progress dialog too! 
 
Thanks again. 
 
Ben. 
Comment 6 Tobias Koenig 2009-08-05 15:59:42 UTC
The development of the old KAddressBook will be discontinued for KDE 4.4.
Since the new application has the same name, but a completly new code base we close all bug reports against the old version and ask the submitters to resend there reports against the new product.