Problem z prędkością wykonywania zapytania, Przepisanie wartości jednej kolumny do drugiej Opcje

[Radek_1]

Witam,

mam problem z prędkością wykonywania zapytania.
Cron uruchamia skrypt codziennie i jeżeli zajdą pewne warunki to przepisuje wartości z jednej kolumny do drugiej. W tabeli jest póki co ok. 37 tyś rekordów. Problem w tym, że przepisywanie trwa bardzo długo, tj. tylko 2tyś rekordów na minutę. Wydawało mi się, że taka nieskomplikowana rzecz powinna trwać parę sekund.
Na początku myślałem, że to wina pythona, dlatego przepisałem to na php, ale wciąż to samo.

Sam kod do przepisywania wygląda tak:

[PHP] pobierz, plaintext 
		$sql="SELECT current_month, name FROM time_online WHERE current_month>0";
		$query = mysql_db_query($db_database, $sql);
		while ($high = mysql_fetch_array($query)) {
			mysql_query("UPDATE time_online SET last_month='".$high[0]."', current_month='0' WHERE name='".$high[1]."'");
		}
[PHP] pobierz, plaintext 

Całość wykonuje się na serwerach home.pl, więc nie powinno być dużych opóźnień. Co o tym sądzicie? Da się to jakoś przyspieszyć, czy taka "prędkość" to norma?


Edit:
Zmieniłem warunek po którym edytujemy wpisy z name na uid i... idzie o niebo szybciej. Taki głupi błąd :/

Ten post edytował Radek_1 1.09.2011, 20:32:16

[mortus]

Nie zrozumiałeś mnie, mysql_affected_rows() używamy, aby uzyskać liczbę rekordów zmienionych przez zapytanie z INSERT, UPDATE, REPLACE i DELETE. Inaczej nie możemy tej liczby uzyskać, bo powyższe zapytania nie zwracają żadnych rekordów.
Natomiast mysql_num_rows() możemy użyć, jeśli chcemy uzyskać liczbę rekordów pobranych za pomocą SELECT. Funkcja ta jest jednak wolna.

[PHP] pobierz, plaintext 
// wariant szybszy$sql = "SELECT COUNT(*) AS `rows` FROM `table`";
$result = mysql_query($sql);
$row = mysql_fetch_assoc($result)
$rowsNumber = $row['rows'];
// wariant wolniejszy
$sql = "SELECT * FROM `table`";
$result = mysql_query($sql);
$rowsNumber = mysql_num_rows($result);
 
// natomiast mysql_affected_rows() używamy dla zapytań z INSERT, UPDATE, REPLACE lub DELETE
// zapytanie aktualizuje czas przebywania online użytkownika o nazwie $name_p
$sql = "UPDATE time_online SET ".$dzien_tyg."=".$dzien_tyg."+15, current_month=current_month+15, online='1' WHERE name='".$name_p."'";
$result = mysql_query($sql);
// sprawdzamy, czy jakieś rekordy zostały zaktualizowane i jeśli nie został zaktualizowany żaden rekord, to poniższy warunek jest spełniony
if(mysql_affected_rows($result) == 0) {
 // co oznacza, że użytkownik o nazwie $name_p jeszcze w naszej tabeli nie istnieje i musimy go dodać
 mysql_query("INSERT INTO time_online SET name='".$name_p."', current_month='15', online='1', ".$dzien_tyg."='15'");
}
[PHP] pobierz, plaintext 

Akurat Twój skrypt to szczególny przypadek, w którym - jak widzisz - nie ma potrzeby sprawdzania za pomocą SELECT, czy użytkownik w tabeli time_online istnieje, wystarczy bowiem spróbować dokonać aktualizacji rekordu, a jeśli aktualizacja się nie powiedzie (mysql_affected_rows() zwróci 0) to będzie to oznaczać, że użytkownika o nazwie $name_p nie ma w tabeli time_online i trzeba go dodać.
Odsyłam do manuala: mysql_num_rows(), mysql_affected_rows().

Skrypt wyszukuje wszystkich użytkowników na stronie i zlicza ich czas bycia online. Jeżeli użytkownik jest nowy, to dodaje go do czasu "obserwowanych", stąd ten INSERT.

Nie zmienia to faktu, że lepiej do wyszukiwania użytkownika nadawałaby się kolumna UID aniżeli name, oraz że obie te kolumny powinny być unikalne i śmiało można nałożyć na nie indeksy, co przyspieszy nieco działanie skryptu.

Ten post edytował mortus 4.09.2011, 09:24:59