Title: [143798] branches/chromium/1410/Source/WebCore/platform/leveldb/LevelDBDatabase.cpp
- Revision
- 143798
- Author
- [email protected]
- Date
- 2013-02-22 14:19:45 -0800 (Fri, 22 Feb 2013)
Log Message
Merge 143512
> IndexedDB: Limit LevelDB's max open files
> https://bugs.webkit.org/show_bug.cgi?id=109993
>
> Reviewed by Tony Chang.
>
> LevelDB keeps up to 1000 (by default) data files open at a time to
> avoid having to open() them. This has caused chromium to hit the
> process-wide open file limit. This patch changes max_open_files to 20,
> as that's the minimum permitted by LevelDB and we have no reason to
> think that performance will suffer because of the extra open calls.
>
> No new tests - a chromium browser test that tracks the open LevelDB
> files is plausible but is blocked on http://crbug.com/177249/.
>
> * platform/leveldb/LevelDBDatabase.cpp:
> (WebCore::openDB):
>
[email protected]
Review URL: https://codereview.chromium.org/12326083
Modified Paths
Diff
Modified: branches/chromium/1410/Source/WebCore/platform/leveldb/LevelDBDatabase.cpp (143797 => 143798)
--- branches/chromium/1410/Source/WebCore/platform/leveldb/LevelDBDatabase.cpp 2013-02-22 22:17:49 UTC (rev 143797)
+++ branches/chromium/1410/Source/WebCore/platform/leveldb/LevelDBDatabase.cpp 2013-02-22 22:19:45 UTC (rev 143798)
@@ -133,6 +133,8 @@
options.comparator = comparator;
options.create_if_missing = true;
options.paranoid_checks = true;
+ // 20 max_open_files is the minimum LevelDB allows.
+ options.max_open_files = 20;
options.env = env;
return leveldb::DB::Open(options, path.utf8().data(), db);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
