differential data management and statistics on data access
authorMatteo Nastasi (mop) <nastasi@alternativeoutput.it>
Sun, 20 Mar 2011 18:00:32 +0000 (19:00 +0100)
committerMatteo Nastasi (mop) <nastasi@alternativeoutput.it>
Sun, 20 Mar 2011 18:00:32 +0000 (19:00 +0100)
web/index_rd.php

index 9b82158..3dec5e9 100644 (file)
@@ -28,6 +28,11 @@ require_once("Obj/brisk.phh");
 // require_once("Obj/proxyscan.phh");
 require_once("briskin5/Obj/briskin5.phh");
 
+$S_load_stat = array( 'U_first_loop' => 0,
+                    'R_garbage'    => 0,
+                    'U_heavy'      => 0,
+                    'R_minusone'   => 0,
+                    'R_the_end'    => 0 );
 
 $mlang_indrd = array( 
                      'btn_backtotab'  => array('it' => ' torna ai tavoli ',
@@ -85,232 +90,257 @@ function page_sync($sess, $page, $table_idx, $table_token)
 
 function maincheck($sess, $cur_stat, $cur_subst, $cur_step, &$new_stat, &$new_subst, &$new_step)
 {
-  GLOBAL $G_lang, $mlang_indrd, $is_page_streaming, $first_loop;
-  GLOBAL $G_with_splash, $G_splash_content, $G_splash_interval, $G_splash_idx;
-  GLOBAL $G_splash_w, $G_splash_h, $G_splash_timeout;
-  $CO_splashdate = "CO_splashdate".$G_splash_idx;
-  GLOBAL $$CO_splashdate;
-  
-  $ret = FALSE;
-  $room = FALSE;
-
-  // log_rd2("M");
-  /* Sync check (read only without modifications */
-  ignore_user_abort(TRUE);
-  if (($sem = Room::lock_data()) != FALSE) { 
-    // Aggiorna l'expire time lato server
-    if  ($first_loop == TRUE) {
-      log_only("F");
-      $room = &Room::load_data();
-      if (($user = &$room->get_user($sess, $idx)) == FALSE) {
-       Room::unlock_data($sem);
-        ignore_user_abort(FALSE);
-       return (unrecerror());
-      }
-      log_auth($sess, "update lacc");
-      $user->lacc = time();
+    GLOBAL $G_lang, $mlang_indrd, $is_page_streaming, $first_loop;
+    GLOBAL $G_with_splash, $G_splash_content, $G_splash_interval, $G_splash_idx;
+    GLOBAL $G_splash_w, $G_splash_h, $G_splash_timeout;
+    $CO_splashdate = "CO_splashdate".$G_splash_idx;
+    GLOBAL $$CO_splashdate;
+    
+    GLOBAL $S_load_stat;
 
-      log_main("pre garbage_manager TRE");
-      $room->garbage_manager(FALSE);
-      
-      Room::save_data($room);
-      $first_loop = FALSE;
-    }
+    log_rd("maincheck begin");
 
-    log_lock("U");
-    Room::unlock_data($sem);
-    ignore_user_abort(FALSE);
-  }
-  else {
-    // wait 20 secs, then restart the xhr 
-    ignore_user_abort(FALSE);
-
-    return ("sleep(gst,20000);|xhr_rd_abort(xhr_rd);");
-    /*
-    ignore_user_abort(FALSE);
-    return (FALSE);
-    */
-  }
+    $ret = FALSE;
+    $room = FALSE;
+    $user = FALSE;
+    $curtime = time();
+    
+    if (($proxy_step = User::step_get($sess)) == FALSE) {
+        log_only2("R");
+        return (FALSE);
+    }
+    
+    // log_rd2("M");
+    /* Sync check (read only without modifications */
+    ignore_user_abort(TRUE);
+    if  ($first_loop == TRUE) {
+        if (($sem = Room::lock_data()) != FALSE) { 
+            // Aggiorna l'expire time lato server
+            $S_load_stat['U_first_loop']++;
+            if (($user = User::load_data($proxy_step['i'], $sess)) == FALSE) {
+                Room::unlock_data();
+                ignore_user_abort(FALSE);
+                return (unrecerror());
+            }
+            $user->lacc = $curtime;
+            User::save_data($user, $user->idx);
+            
+            if (Room::garbage_time_is_expired($curtime)) {
+                log_only("F");
+                
+                $S_load_stat['R_garbage']++;
+                if (($room = Room::load_data()) != FALSE) {
+                    log_main("pre garbage_manager TRE");
+                    $room->garbage_manager(FALSE);
+                    Room::save_data($room);
+                }
+            }
+            log_lock("U");
+            Room::unlock_data($sem);
+            ignore_user_abort(FALSE);
+        } // if (($sem = Room::lock_data()) != FALSE) { 
+        else {
+            // wait 20 secs, then restart the xhr 
+            ignore_user_abort(FALSE);
+            
+            return ("sleep(gst,20000);|xhr_rd_abort(xhr_rd);");
+        }
+        $first_loop = FALSE;
+    } // if  ($first_loop == TRUE) {
     
-  if (($proxy_step = step_get($sess)) != FALSE) {
-    // log_rd2("Postget".$proxy_step."zizi");
-
     if ($cur_step == $proxy_step['s']) {
-      log_lock("P");
-      return (FALSE);
+        log_lock("P");
+        return (FALSE);
     }
     else {
-      log_only2("R");
+        log_only2("R");
     }
-  }
-  else {
-      log_only2("R");
-  }
-
-  if ($room == FALSE) {
-    do {
-      ignore_user_abort(TRUE);
-      if (($sem = Room::lock_data()) == FALSE) 
-       break;
-      
-      log_lock("P");
-      if (($room = &Room::load_data()) == FALSE) 
-       break;
-    } while (0);
     
-    if ($sem != FALSE)
-      Room::unlock_data($sem);
-    
-    ignore_user_abort(FALSE);
-    if ($room == FALSE) 
-      return (FALSE);
-  }
-  
-  if (($user = &$room->get_user($sess, $idx)) == FALSE) {
-    return (unrecerror());
-  }
-
-  /* Nothing changed, return. */
-  if ($cur_step == $user->step) 
-    return (FALSE);
 
-  log_rd2("do other ++".$cur_stat."++".$user->stat."++".$cur_step."++".$user->step);
+    if (1 == 1) { // C'è veramente qualche caso in cui serve questo step ??
+        if ($user == FALSE) {
+            do {
+                ignore_user_abort(TRUE);
+                if (($sem = Room::lock_data()) == FALSE) 
+                    break;
+                
+                log_lock("P");
+                $S_load_stat['U_heavy']++;
+                if (($user = User::load_data($proxy_step['i'], $sess)) == FALSE) {
+                    break;
+                }
+            } while (0);
+            
+            if ($sem != FALSE)
+                Room::unlock_data($sem);
+            
+            ignore_user_abort(FALSE);
+            if ($user == FALSE) {
+                return (unrecerror());
+            }
+        }
+        
+        /* Nothing changed, return. */
+        if ($cur_step == $user->step) 
+            return (FALSE);
+    } // if (0 == 1) { xx C'è veramente qualche caso in cui serve questo step ??
 
-  if ($cur_step == -1) {
-    // FUNZIONE from_scratch DA QUI 
-    ignore_user_abort(TRUE);
-    $sem = Room::lock_data();
-    $room = &Room::load_data();
-    if (($user = &$room->get_user($sess, $idx)) == FALSE) {
-      Room::unlock_data($sem);
-      ignore_user_abort(FALSE);
-      return (unrecerror());
-    }
-    if ($user->the_end) { 
-      log_rd2("main_check: the end".var_export(debug_backtrace()));
-      $is_page_streaming = TRUE;
+    
+    log_rd2("do other cur_stat[".$cur_stat."] user->stat[".$user->stat."] cur_step[".$cur_step."] user_step[".$user->step."]");
+    
+    if ($cur_step == -1) {
+        /*
+         *  if $cur_step == -1 load the current state from the main struct
+         */
+        ignore_user_abort(TRUE);
+        $sem = Room::lock_data();
+        $room = Room::load_data();
+        $S_load_stat['R_minusone']++;
+        
+        /* unset the $user var to reload it from main structure */
+        unset($user);
+        if (($user = $room->get_user($sess, $idx)) == FALSE) {
+            Room::unlock_data($sem);
+            ignore_user_abort(FALSE);
+            return (unrecerror());
+        }
+        
+        if ($user->the_end) { 
+            log_rd2("main_check: the end".var_export(debug_backtrace()));
+            $is_page_streaming = TRUE;
+        }
+        
+        if ($user->trans_step != -1) {
+            log_rd2("TRANS USATO ".$user->trans_step);
+            $cur_step = $user->trans_step;
+            $user->trans_step = -1;
+            
+            Room::save_data($room);
+            Room::unlock_data($sem);
+            ignore_user_abort(FALSE);
+        }
+        else {
+            log_rd2("TRANS NON ATTIVATO");
+            //        ARRAY_POP DISABLED
+            //        log_rd2("TRANS NON ATTIVATO, clean del comm array");
+            //        while (($el = array_pop($user->comm)) != NULL) { 
+            //          log_rd2("clean element [".$el."]");
+            //        }
+            //        //        $user->step_inc(COMM_N + 1);
+            //        Room::save_data($room);
+            //        //        $new_step = $user->step;
+            
+            Room::unlock_data($sem);
+            ignore_user_abort(FALSE);
+        }
     }
-
-    if ($user->trans_step != -1) {
-      log_rd2("TRANS USATO ".$user->trans_step);
-      $cur_step = $user->trans_step;
-      $user->trans_step = -1;
-
-
-      Room::save_data($room);
-      Room::unlock_data($sem);
-      ignore_user_abort(FALSE);
+    
+    
+    /* this part I suppose is read only on $room structure */
+    if ($cur_step == -1) {
+        log_rd2("PRE-NEWSTAT: ".$user->stat);
+        
+        if ($user->stat == 'room') {
+            log_rd("roomma ".$user->step);
+            $curtime = time();
+            
+            if ($G_with_splash &&
+                ($$CO_splashdate < $curtime - $G_splash_interval ||
+                 $$CO_splashdate > $curtime)) {
+                $is_super = $user->flags & USER_FLAG_TY_SUPER;
+                $ret .=  show_notify_ex(str_replace("\n", " ", $G_splash_content[$G_lang]), 
+                                        ($is_super ? 0 : $G_splash_timeout), 
+                                        $mlang_indrd[($is_super ? 'btn_btotabsup' : 'btn_backtotab')][$G_lang], 
+                                        $G_splash_w, $G_splash_h, true, 
+                                        ($is_super ? 0 : $G_splash_timeout));
+                $ret .= sprintf('|createCookie("CO_splashdate%d", %d, 24*365, cookiepath);', $G_splash_idx, $curtime);
+            }
+            $ret .= $room->show_room($user->step, &$user);
+            
+            // TODO uncomment and test
+            /* NOTE the sets went common */
+            $new_stat =  $user->stat;
+            $new_subst = $user->subst;
+            $new_step =  $user->step;
+        }
+        /***************
+         *             *
+         *    TABLE    *
+         *             *
+         ***************/
+        else if ($user->stat == 'table') {
+            log_load("RESYNC");
+            return (page_sync($user->sess, "briskin5/index.php", $user->table, $user->table_token));
+        }
+        log_rd2("NEWSTAT: ".$user->stat);
     }
     else {
-       log_rd2("TRANS NON ATTIVATO");
-//        ARRAY_POP DISABLED
-//        log_rd2("TRANS NON ATTIVATO, clean del comm array");
-//        while (($el = array_pop($user->comm)) != NULL) { 
-//          log_rd2("clean element [".$el."]");
-//        }
-//        //        $user->step_inc(COMM_N + 1);
-//        Room::save_data($room);
-//        //        $new_step = $user->step;
-        
-       Room::unlock_data($sem);
-       ignore_user_abort(FALSE);
-    }
-  }
-      
-  if ($cur_step == -1) {
-    log_rd2("PRE-NEWSTAT: ".$user->stat);
-
-    if ($user->stat == 'room') {
-      log_rd("roomma ".$user->step);
-      $curtime = time();
-
-      if ($G_with_splash &&
-          ($$CO_splashdate < $curtime - $G_splash_interval ||
-           $$CO_splashdate > $curtime)) {
-          $is_super = $user->flags & USER_FLAG_TY_SUPER;
-          $ret .=  show_notify_ex(str_replace("\n", " ", $G_splash_content[$G_lang]), 
-                                  ($is_super ? 0 : $G_splash_timeout), 
-                                  $mlang_indrd[($is_super ? 'btn_btotabsup' : 'btn_backtotab')][$G_lang], 
-                                  $G_splash_w, $G_splash_h, true, 
-                                  ($is_super ? 0 : $G_splash_timeout));
-        $ret .= sprintf('|createCookie("CO_splashdate%d", %d, 24*365, cookiepath);', $G_splash_idx, $curtime);
+        // TODO: verify if we can use only $user struct 
+        ignore_user_abort(TRUE);
+        $sem = Room::lock_data();
+        $S_load_stat['U_heavy']++;
+        if (($user = User::load_data($proxy_step['i'], $sess)) == FALSE) {
+            Room::unlock_data();
+            ignore_user_abort(FALSE);
+            return (unrecerror());
+        }
+        
+        if ($cur_step < $user->step) {
+            do {
+                if ($cur_step + COMM_N < $user->step) {
+                    if (($cur_stat != $user->stat)) {
+                        $to_stat = $user->stat;
+                        Room::unlock_data($sem);
+                        ignore_user_abort(FALSE);
+                        log_load("RESYNC");
+                        return (page_sync($user->sess, ($to_stat == "table" ? "briskin5/index.php" : "index.php"), $user->table, $user->table_token));
+                    }
+                    log_rd2("lost history, refresh from scratch");
+                    $new_step = -1;
+                    break;
+                } 
+                for ($i = $cur_step ; $i < $user->step ; $i++) {
+                    log_rd2("ADDED TO THE STREAM: ".$user->comm[$i % COMM_N]);
+                    $ret .= $user->comm[$i % COMM_N];
+                }
+                $new_stat =  $user->stat;
+                $new_subst = $user->subst;
+                $new_step =  $user->step;
+            } while (0);
+            
+            log_mop($user->step, 'index_rd.php: after ret set');
+            
+          if ($user->the_end == TRUE) {
+              log_rd2("LOGOUT BYE BYE!!");
+              log_auth($user->sess, "Explicit logout.");
+              
+              $S_load_stat['R_the_end']++;
+              $room = &Room::load_data();
+              unset($user);
+              if (($user = &$room->get_user($sess, $idx)) == FALSE) {
+                  Room::unlock_data($sem);
+                  ignore_user_abort(FALSE);
+                  return (unrecerror());
+              }              
+              $user->reset();
+              
+              if ($user->subst == 'sitdown') {
+                  log_load("ROOM WAKEUP");
+                  $room->room_wakeup($user);
+              }
+              else if ($user->subst == 'standup')
+                  $room->room_outstandup(&$user);
+              else
+                  log_rd2("LOGOUT FROM WHAT ???");
+              
+              Room::save_data($room);
+          }
       }
-      $ret .= $room->show_room($user->step, &$user);
-
-      // TODO uncomment and test
-      /* NOTE the sets went common */
-      $new_stat =  $user->stat;
-      $new_subst = $user->subst;
-      $new_step =  $user->step;
-    }
-    /***************
-     *             *
-     *    TABLE    *
-     *             *
-     ***************/
-    else if ($user->stat == 'table') {
-      log_load("RESYNC");
-      return (page_sync($user->sess, "briskin5/index.php", $user->table, $user->table_token));
-    }
-    log_rd2("NEWSTAT: ".$user->stat);
-  }
-  else {
-    ignore_user_abort(TRUE);
-    $sem = Room::lock_data();
-    $room = &Room::load_data();
-    if (($user = &$room->get_user($sess, $idx)) == FALSE) {
+      
       Room::unlock_data($sem);
       ignore_user_abort(FALSE);
-      return (unrecerror());
-    }
-    if ($cur_step < $user->step) {
-      do {
-       if ($cur_step + COMM_N < $user->step) {
-         if (($cur_stat != $user->stat)) {
-           $to_stat = $user->stat;
-           Room::unlock_data($sem);
-           ignore_user_abort(FALSE);
-           log_load("RESYNC");
-           return (page_sync($user->sess, ($to_stat == "table" ? "briskin5/index.php" : "index.php"), $user->table, $user->table_token));
-         }
-         log_rd2("lost history, refresh from scratch");
-          $new_step = -1;
-         break;
-       } 
-       for ($i = $cur_step ; $i < $user->step ; $i++) {
-         log_rd2("ADDED TO THE STREAM: ".$user->comm[$i % COMM_N]);
-         $ret .= $user->comm[$i % COMM_N];
-       }
-       $new_stat =  $user->stat;
-       $new_subst = $user->subst;
-       $new_step =  $user->step;
-      } while (0);
-
-      log_mop($user->step, 'index_rd.php: after ret set');
-
-      if ($user->the_end == TRUE) {
-       log_rd2("LOGOUT BYE BYE!!");
-       log_auth($user->sess, "Explicit logout.");
-
-       $user->reset();
-
-       if ($user->subst == 'sitdown') {
-         log_load("ROOM WAKEUP");
-         $room->room_wakeup($user);
-       }
-       else if ($user->subst == 'standup')
-         $room->room_outstandup(&$user);
-       else
-         log_rd2("LOGOUT FROM WHAT ???");
-         
-       Room::save_data($room);
-      }
-    }
-         
-    Room::unlock_data($sem);
-    ignore_user_abort(FALSE);
   }
-
+  
   
   return ($ret);
 }
@@ -370,6 +400,13 @@ for ($i = 0 ; time() < $endtime ; $i++) {
     echo '_';
     flush();
   }
+ }
+
+$s = "[".$sess."] index_rd.php stats: ";
+foreach ($S_load_stat as $key => $value) {
+    $s .= sprintf("%s: %d - ", $key, $value);
 }
+error_log($s, 0);
+
 
 ?>