Performance - order_history.php

[Taste]

Guten Abend,

ich weiß ich habe mich noch nicht vorgestellt, ich werde das später nachholen, aber wenn ich jetzt nicht gleich schreibe, vergesse ich es wieder

Kurze Vorgeschichte bevor ich zu dem Problem kommen:
Wir haben heute ein neues Produkt eingeführt, zum Verkaufsstart waren ~1000 Kunden auf der Seite und wollten einkaufen. Dann ist der Mysql Server abgestürzt. (Server: 2 x Intel® Xeon® 2,13 GHz, 32GB Arbeitsspeicher, Mysql läuft auf SSD)

Heute Abend kam ich dazu die Slow-Query-Logs auszuwerten und habe erstmal gestaunt (siehe Anhang.)

Code: [Auswählen]

select distinct op.products_id from orders o, orders_products op, products p where o.customers_id = '****' and o.orders_id = op.orders_id and op.products_id = p.products_id and p.products_status = '1' group by products_id order by o.date_purchased desc limit 6

Das ist die einzige Zeile, die die Slow Query Log füllt. Ausgeführt wird diese in der order_history.php:

templates/******/source/boxes/order_history.php:

Code: [Auswählen]

// retreive the last x products purchased
    $orders_query = xtc_db_query("select distinct op.products_id from " . TABLE_ORDERS . " o, " . TABLE_ORDERS_PRODUCTS . " op, " . TABLE_PRODUCTS . " p where o.customers_id = '" . (int)$_SESSION['customer_id'] . "' and o.orders_id = op.orders_id and op.products_id = p.products_id and p.products_status = '1' group by products_id order by o.date_purchased desc limit " . MAX_DISPLAY_PRODUCTS_IN_ORDER_HISTORY_BOX);

Das Ergebnis wird an das Template in "box_HISTORY" übergeben, so können einem Kunden die letzten 6 Produkte ausgegeben werden, die er bestellt hat.

Eingebunden wird das ganze in:

templates/******/source/boxes.php:

Code: [Auswählen]

  if (isset($_SESSION['customer_id'])) include(DIR_WS_BOXES . 'order_history.php');

Verwendung findet das ganze keine, also nicht nur bei mir auch im XTC Template wird es nicht verwendet, oder habe ich etwas übersehen?
Für mich sieht das ganze nach Altlast aus, vielleicht sollte das ganze im nächsten Update entfernt werden? (Ich hoffe ich habe auf die Schnelle nichts übersehen)

Die Überlastung des Servers hatte noch ein Paar andere Nebeneffekte, dazu werde ich morgen noch etwas schreiben, ist ja eine Situation, die sich schlecht simulieren lässt

Viele Grüße und noch einen schönen Abend,
Steffen


Linkback: https://www.modified-shop.org/forum/index.php?topic=20884.0

[Modulfux]

Hallo Steffen,

mit deiner Aussage hast du völlig recht. Die Box "order_history" ist ein wenig obsolet.
Die Box selber wird in der index.html vom Template nicht aufgerufen. Man könnte dieses durch {$box_HISTORY} machen.

Aber wie ich schon erwähnt habe, ist diese Box in meinen Augen obsolet, zumal der eingeloggte Kunde auch innerhalb des Bereiches "Mein Konto" seine letzten Bestellungen sehen kann.

Gruß
Ronny

[Taste]

Ich hab die order_history.php inzwischen in der boxes.php auskommentiert. Kann ich nur empfehlen.
Im eingeloggten Zustand hat sich die Ladezeit des Shops um eine gefühlte Sekunde verringert (im Frontend).

Das Query ist ja auch nicht ohne, das läuft über 3 Tabellen + distinct + group by

[DokuMan]

Stimmt, das {$box_HISTORY} wird nie wirklich verwendet... ich seh das heute zum ersten Mal 

Ich denke auch, das kann raus, zumal wie gesagt unter "Mein Konto" alles schön dargestellt ist und man es da auch eher vermutet.

[Simon]

Ich hab die order_history.php inzwischen in der boxes.php auskommentiert. Kann ich nur empfehlen.

Kann man es wirklich gefahrlos löschen oder wird es doch für irgendetwas benötigt ?

[jannemann]

Hallo Simon,

du kannst es löschen, ich empfehle es aber auszukommentieren.
Das kannst du gefahrlos machen.

Schöne Grüße,
Jan

[Simon]

Hallo Jan,

danke dir. Im Trunk ist es noch nicht auskommentiert, jetzt war ich mir nicht sicher.

[Taste]

Wollte noch einmal hierauf hinweisen. In der aktuellen Version die zum Download angeboten wird besteht das Problem noch immer. Ist sicher untergegangen