Just think it would be nice to have pagination. There are thousands of pages with performance issues. It's going to be a pretty long list....

EDIT: This is primarily for the summary page.

Comments

malc0mn’s picture

That is a bit of a problem indeed, especially when the data comes from APC, Memcache or Zendcache. Thought about how to do this, but no working solution just yet...

mgifford’s picture

No way to build a temporary table & then sort through that? It did occur to me that this might be difficult to pull off.

It isn't urgent but occurred to be as being a useful thing.

malc0mn’s picture

Creating a temp table would be fairly easy IMHO, but would totally ruin the implementation of APC / Memcache / Zend and other caching layers for live sites. If you would dump the data to DB when viewing the summary pages, that would cause load that you do not want on a live site.

Unless you would do it on cron in the middle of the night, and even then...

I have code in place for the paging for quite some time (works like a charm for DB data) but my attempts were not succesfull for a non-db source. The todo's are in the code:

  global
    $pager_page_array,  // array of element-keyed current page - 1
    $pager_total,       // array of element-keyed total number of pages
    $pager_total_items, // array of element-keyed total number of data rows
    $pager_limits;      // array of element-keyed number of rows per page

  [...]

  $pager_height = 50;

  [...]

  // TODO: make this work properly!
  // Set up pager since this is not done automatically when not using DB.
  if ($source != 'db' && $data_list) {
    $page = isset($_GET['page']) ? sprintf('%d', $_GET['page']) : 0;

    $pager_page_array  = array(0 => $page);
    $pager_total_items = array(0 => count($data_list));
    $pager_limits      = array(0 => $pager_height);
    $pager_total       = array(0 => ceil($pager_total_items[0] / $pager_limits[0]));

    // Extract the data subset we need.
    $data_list = array_slice($data_list, $page * $pager_height, $pager_height);
  }

:-D

malc0mn’s picture

Been testing and summary list paging should work like a charm. How many entries do you have? What cache store are you logging to?

mgifford’s picture

There were thousands, but we have to turn it off now that it's in production. Will set it up on the staging servers soon though.

Should be hundreds of links I'd think just using the selenium testing.

mgifford’s picture

Issue summary: View changes

clarification on detail/summary.