| 1 |
On 10/23/2014 02:22 AM, Zac Medico wrote: |
| 2 |
> In the future, we might decide to rewrite the search class so that it |
| 3 |
> processes the index as a stream, which will allow individual search |
| 4 |
> results to be displayed as soon as they are located [1]. This rewrite |
| 5 |
> will require a new index API. |
| 6 |
> |
| 7 |
> So, considering that we probably want a new index API in the future, we |
| 8 |
> might save some unnecessary effort and stick with the |
| 9 |
> special-purpose/minimalistic IndexedPortdb/Vardb adapters for now. |
| 10 |
> |
| 11 |
> [1] https://bugs.gentoo.org/show_bug.cgi?id=412471 |
| 12 |
|
| 13 |
I have a stream-based back end working in this branch: |
| 14 |
|
| 15 |
https://github.com/zmedico/portage/commits/bug_525718_stream |
| 16 |
|
| 17 |
Now all that's left to do is to convert IndexedPortdb to expose an |
| 18 |
iter_cp_all method, and to fix the search class to iterate over |
| 19 |
iter_cp_all and display results incrementally. Once that's done, I'll |
| 20 |
think about exposing useful pieces as public APIs. |
| 21 |
-- |
| 22 |
Thanks, |
| 23 |
Zac |
Archives