Reloads löschen sich nicht aus der DB :( VMS 1.2

[Gremlin]

Eleganter wäre es nur die wirklich abgelaufenen Reloads zu löschen da manche auch ggf. länger als 48 Stunden sind.

<?php
db_query('DELETE FROM '.$db_prefix.'_reloads WHERE bis <= '.time().'') or die('Löschen fehlgeschlagen '.mysql_error());
db_query('OPTIMIZE NO_WRITE_TO_BINLOG TABLE '.$db_prefix.'_reloads') or die('Optimierung fehlgeschlagen '.mysql_error());
?>

Ausserdem: time() ist kein String sondern ein Integer deshalb sind die doppelten Anführungszeichen falsch.

Ausserdem statt db_query(""); einfach db_query(''); nutzen dann muss php nicht nochmal den kompletten SQL Befehl parsen, beschleunigt das ganze auch nochmal minimal :smilewinkgrin:

Da beim Reloads löschen viele Datensätze gelöscht werden sollte die Tabelle danach optimiert werden

Gruß
Gremlin

[didith1207]

aja vielen dank

[SebbyPHM]

Eleganter wäre es nur die wirklich abgelaufenen Reloads zu löschen da manche auch ggf. länger als 48 Stunden sind.

<?php
db_query('DELETE FROM '.$db_prefix.'_reloads WHERE bis <= '.time().'') or die('Löschen fehlgeschlagen '.mysql_error());
db_query('OPTIMIZE NO_WRITE_TO_BINLOG TABLE '.$db_prefix.'_reloads') or die('Optimierung fehlgeschlagen '.mysql_error());
?>

Ausserdem: time() ist kein String sondern ein Integer deshalb sind die doppelten Anführungszeichen falsch.

Ausserdem statt db_query(""); einfach db_query(''); nutzen dann muss php nicht nochmal den kompletten SQL Befehl parsen, beschleunigt das ganze auch nochmal minimal :smilewinkgrin:

Da beim Reloads löschen viele Datensätze gelöscht werden sollte die Tabelle danach optimiert werden

Gruß
Gremlin

fängst ja so langsam an, wie manchmal ich zu klingen :eek6:

[Gremlin]

fängst ja so langsam an, wie manchmal ich zu klingen :eek6:

Ja klar *grins* soll ja immerhin ordentlich werden.

[wolfi86]

Für die Leute, bei denen nichts passiert, denkt bitte daran, dass ihr erst eine Verbindung zur Datenbank aufbauen müsst
Das sieht dann so au:

<?php
@require_once ('../lib/functions.lib.php');
db_connect();
db_query('DELETE FROM '.$db_prefix.'_reloads WHERE bis <= '.time().'') or die('Löschen fehlgeschlagen '.mysql_error());
db_query('OPTIMIZE NO_WRITE_TO_BINLOG TABLE '.$db_prefix.'_reloads') or die('Optimierung fehlgeschlagen '.mysql_error());
db_close();
echo'VMS_Reloads erfolgreich optimiert';
?>

[Xenon]

@wolfi : das ist für leute die das vms1.1 nutzen was gremlin da gemacht hat ist für vms1,2

[dragon11]

Frage: warum einen extra cron????
Habt ihr das nicht im cron db_opzimize.php drin? dieser solle doch eh stündlich laufen und die beiden befehle da noch unterzubringen, ist ja keine Hürde.

[jpwfour]

man kann auch einfach in die header.php folgendes einbauen:

db_query ("DELETE FROM ".$db_prefix."_reloads WHERE bis <= '".time()."'"); 

wer aber unbedingt db anfragen sparen will kann das ganze ja noch so verändern:

if(rand(0,4)==2) db_query ("DELETE FROM ".$db_prefix."_reloads WHERE bis <= '".time()."'"); 

könnte vielleicht sogar besser sein, weil so immer mal wenige zeilen gelöscht werden, beim cron aufruf aber auf einmal hunderte, nur sollte natürlich ein optimize befehl dennoch immer mal wieder ausgeführt werden.

[didith1207]

Frage: warum einen extra cron????
Habt ihr das nicht im cron db_opzimize.php drin? dieser solle doch eh stündlich laufen und die beiden befehle da noch unterzubringen, ist ja keine Hürde.

db_opzimize.php hab noch niemals von der datei gehört wo sollte ich die finden?