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

[didith1207]

Hallo ich habe das VMS 1.2

ich habe mal meine Loseseite genauer unter die lupe genommen und festgestellt das in der datenbank unter *vms_reloads* über 400k abgelaufene reloads drinnen stehen die schon minimum 1 monat abgelaufen sind manche davon sogar 3 monate
Zeige Datensätze 487930 - 487948

Wo kann das problem liegen löschen die sich nicht automatisch?

[pummuk]

es gab mal ein cron hier... war aber für´s vms1.1

<?
$timel = time() - 172800;
@require_once ('../lib/functions.lib.php');
db_connect();
db_query("DELETE FROM `vms_reloads` WHERE `bis` <='".$timel."'");
db_close();
echo'VMS_Reloads erfolgreich optimiert';
?>

weiß nicht wie es für´s vms 1.2 lauten müßte...

[didith1207]

Vielen dank! ich habs nun so gelöst :

ich hab mir einen cron fürs VMS 1.2 erstellt mit dem namen reloadcron.php:

<?
$timel = time() - 172800;
db_query("DELETE FROM `vms_reloads` WHERE `bis` <='".$timel."'");
echo'VMS_Reloads erfolgreich optimiert';
?>

dann habe ich für die db den cron eingefügt :
INSERT INTO `vms_crons` VALUES ('','ext_reloadcron', 0, 'crons/reloadcron.php', 'Reloads Loeschen');

Beim abrufen ( dauerte einige sekunden da er ja über 400k einträge löschen mußte) bekam ich folgende meldung :

VMS_Reloads erfolgreich optimiertCron gelaufen

und siehe da in der DB wurden alle einträge in vms_reloads die länger als 48h abgelaufen waren gelöscht

Kann ich das so lassen? ist das eine feine lösung?

[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.