Jest jakakolwiek możliwość, zby wykonało się 1 zapytanie, a drugie nie? (zakładając, że dane w zmiennych są poprawne)

[PHP] pobierz, plaintext 
					mysql_query('BEGIN', $linkCron1);
					// Dodaj nową platnosc
					$query2 = "INSERT INTO transaction_pay (transaction_pay.pay_trans_id, transaction_pay.pay_trans_it_id, transaction_pay.pay_trans_count) VALUES ('".$as_result[$a]->{'deal-transaction-id'}."', '".$as_result[$a]->{'deal-item-id'}."', '".$as_result[$a]->{'deal-quantity'}."')";
					if (!mysql_query($query2, $linkCron1)){
						mysql_query('ROLLBACK', $linkCron1);
					}
					else {
						//  Dodaj informacje o opłaconych towarach
						$query2 = "UPDATE transactions SET transactions.pay_quantity=transactions.pay_quantity+".$as_result[$a]->{'deal-quantity'}.", transactions.data_platnosci='".date("Y-m-d H:i:s",$as_result[$a]->{'deal-event-time'})."', transactions.data_przyjeciaplatnosci='".date("Y-m-d H:i:s", $data_teraz)."', transactions.error='0' WHERE transactions.transaction_user_id='".$as_result[$a]->{'deal-buyer-id'}."' AND transactions.allegro_id='".$as_result[$a]->{'deal-item-id'}."' AND transactions.pay_quantity<transactions.buy_quantity";
						if (!mysql_query($query2, $linkCron1)){
							mysql_query('ROLLBACK', $linkCron1);
						}
						else {
							mysql_query('COMMIT', $linkCron1);
						}
					}
[PHP] pobierz, plaintext 

Ponieważ raz na jakiś czas dodaje mi transakcję (insert into transaction_pay...), ale już nie aktualizuje ilości towarów (update transactions ...)
Wspomnę tylko, że ten kawałek kodu jest dość często uruchamiany (cronem co 3minuty a następnie w pętli po kilkaset razy).

Gdzie szukać problemu?

try/catch + logowanie błędów i wszystko będzie jasne.

ps. po co ci dwa rolback'i?

Na jakim silniku sa tabele w bazie? Transakcje nie dzialaja na MyISAM.

btw. Twoj kod jest kompletnie bez sensu, po co tworzyc transakcje dla 1 zapytania?

Bo jeśli jedno zapytanie się nie uda to -> cofam. Jeśli jedno się uda a drugie nie, też -> cofam (drugim rollback).
Nie tak powinno się to robić?


Są na InnoDB


Jakiego jednego? Są dwa: INSERT i UPDATE. Nie powinno zrobić UPDATE, jeśli nie zrobi INSERT i na odwrót.
W takim razie jak to inaczej napisać?

Przejść na PDO i

[PHP] pobierz, plaintext 
try {
begin;
zapytania;
} catch (PDOException $e) {
rollback;
}
[PHP] pobierz, plaintext 

Tak się zastanawiam czy po prostu nie możesz sprawdzić czy zapytanie się wykonało i jeśli tak to wykonujesz drugie?

Transakcje wydawało mi się, że służą troszkę bardziej rozbudowanym celom - np. dodawanie tysiąca rekordów - jeden się nie doda to cofasz i naprawiasz.

Tak robiłem do tej pory, ale czasami zdarzało się, że dodawało do 1 tabeli, ale nie aktualizowało w 2 tabeli.
Dlatego pomyślałem o transakcjach.

Mam jeszcze jeden pomysł.
Bo ten kod jest uruchamiany w cronie i bardzo często używany (setki razy wykonywany co kilka minut).
Na drugiej tabeli jest wykonywanych znacznie więcej operacji, również przez inny cron.

Może po prostu jest natłok operacji na danej bazie i czasami operacje nie są wykonywana?

Jest na to jakaś metoda? Czy blokowanie tablic jest odpowiednie? (nie chciałbym zeby reszta zapytań w tym czasie jak wykonuje jedno to się nie wykonywała lub zwracała błąd, tylko oczekiwała na swoją kolej)

@lukaskolista, @markonix: Każda seria zapytań (włączając w to pojedyncze zapytania) musi być objęta transakcją. W przypadku pojedynczych zapytań można sobie co najwyżej darować jawne deklarowanie transakcji, bo domyślnie włączony jest autocommit.

@jakal: Przede wszystkim przydała by się informacja o wystąpieniu błędu. Poza rollbackiem mógłbyś też zapisywać informację o przyczynie niepowodzenia operacji - ułatwiłoby to Ci pracę. Dodatkowo, nie muszę chyba wspominać o tym, że użycie PDO znacznie uprościłoby kod.

Jakiego typu są to operacje? Mogą one nie mieć kompletnie żadnego znaczenia, być może wystarczy jedynie zwiększyć poziom izolacji transakcji, a być może rzeczywiście trzeba będzie blokować całą tabelę.

EDIT: Sam kod, mimo iż tragicznej jakości, wydaje się być w porządku. Problem jest prawdopodobnie gdzie indziej.

Tak chyba zrobię, jeśli drugie zapytanie się nie wykona to zapiszę czemu, razem z treścią zapytania.
Wiem, że PDO by ułatwiło, ale aplikacja ma za dużo kodu, żeby teraz zmieniać wszytko na PDO.



W większości SELECT, ale też często UPDATE.


Czemu tragiczna jakość? Po prostu czysty PHP + MYSQL, nie obiektowy.

Powinieneś zapisywać informację o wystąpieniu dowolnego, nawet nieistotnego błędu i najlepiej od razu przerywać działanie całej aplikacji w przypadku jego wystąpienia. Kod, który generuje nieobsłużone błędy właściwie z definicji działa w sposób nieokreślony.
Nie da się teraz stwierdzić czy te zapytania mają w ogóle jakikolwiek związek z problemem, ale być może zapytania SELECT operują na innym niż przewidywanym zestawie rekordów - skutek modyfikowania tabeli przez inny proces. Poczytaj o wspomnianych wcześniej poziomach izolacji transakcji, to będziesz wstanie stwierdzić czy to może powodować błąd.
Brak obsługi błędów, nie korzystanie z preparowanych zapytań w momencie gdy to samo zapytanie jest wykonywane setki razy z rzędu, brak odpowiedniego przygotowania danych do wrzucenia w treść zapytania SQL, niepotrzebnie wiele poziomów zagłębienia kodu, niepotrzebne powtarzanie fragmentów kodu, nieczytelne nazwy zmiennych/formatowanie kodu... to tak na pierwszy rzut oka.