Updated footer comment
[metaproxy-moved-to-github.git] / src / filter_load_balance.cpp
index 64536f2..a534f60 100644 (file)
@@ -1,8 +1,20 @@
-/* $Id: filter_load_balance.cpp,v 1.2 2007-01-03 15:03:55 marc Exp $
-   Copyright (c) 2005-2006, Index Data.
+/* This file is part of Metaproxy.
+   Copyright (C) 2005-2008 Index Data
 
-   See the LICENSE file for details
- */
+Metaproxy is free software; you can redistribute it and/or modify it under
+the terms of the GNU General Public License as published by the Free
+Software Foundation; either version 2, or (at your option) any later
+version.
+
+Metaproxy is distributed in the hope that it will be useful, but WITHOUT ANY
+WARRANTY; without even the implied warranty of MERCHANTABILITY or
+FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
+for more details.
+
+You should have received a copy of the GNU General Public License
+along with this program; if not, write to the Free Software
+Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+*/
 
 #include "config.hpp"
 #include "session.hpp"
 #include "filter_load_balance.hpp"
 #include "util.hpp"
 
+
 #include <boost/thread/mutex.hpp>
-#include <boost/date_time/posix_time/posix_time.hpp>
 
 #include <yaz/zgdu.h>
 
-#include <iostream>
+// remove max macro if already defined (defined later in <limits>)
+#ifdef max
+#undef max
+#endif
+
+//#include <iostream>
 #include <list>
 #include <map>
 #include <limits>
@@ -54,11 +71,11 @@ namespace metaproxy_1 {
                 unsigned int deads;
                 unsigned int cost() {
                     unsigned int c = sessions + packages + deads;
-                    std::cout << "cost  c:" << c 
-                              << " s:" << sessions 
-                              << " p:" << packages 
-                              << " d:" << deads 
-                              <<"\n";
+                    //std::cout << "stats  c:" << c 
+                    //          << " s:" << sessions 
+                    //          << " p:" << packages 
+                    //          << " d:" << deads 
+                    //          <<"\n";
                     return c;
                 }
             };
@@ -81,7 +98,7 @@ yf::LoadBalance::~LoadBalance()
 {  // must have a destructor because of boost::scoped_ptr
 }
 
-void yf::LoadBalance::configure(const xmlNode *xmlnode)
+void yf::LoadBalance::configure(const xmlNode *xmlnode, bool test_only)
 {
     m_p->configure(xmlnode);
 }
@@ -112,6 +129,7 @@ void yf::LoadBalance::Impl::process(mp::Package &package)
 {
 
     bool is_closed_front = false;
+    bool is_closed_back = false;
 
     // checking for closed front end packages
     if (package.session().is_closed()){
@@ -167,7 +185,7 @@ void yf::LoadBalance::Impl::process(mp::Package &package)
                 
                 // copying new target into init package
                 mp::util::set_vhost_otherinfo(&(org_init->otherInfo), 
-                                              odr_en, target); 
+                                              odr_en, target, 1);
                 package.request() = gdu_req;
         }
             
@@ -190,15 +208,8 @@ void yf::LoadBalance::Impl::process(mp::Package &package)
 
 
     // checking for closed back end packages
-    if (package.session().is_closed()) {
-        boost::mutex::scoped_lock scoped_lock(m_mutex);
-
-        // marking backend dead if backend closed without fronted close 
-        if (is_closed_front == false)
-            add_dead(package.session().id());
-
-        remove_session(package.session().id());
-    }
+    if (package.session().is_closed())
+        is_closed_back = true;
 
     Z_GDU *gdu_res = package.response().get();
 
@@ -207,14 +218,9 @@ void yf::LoadBalance::Impl::process(mp::Package &package)
 
         // session closing only on Z39.50 close response
         if (gdu_res->u.z3950->which == Z_APDU_close){
+            is_closed_back = true;
             boost::mutex::scoped_lock scoped_lock(m_mutex);
             remove_package(package.session().id());
-            
-            // marking backend dead if backend closed without fronted close 
-            if (is_closed_front == false)
-                add_dead(package.session().id());
-            
-            //remove_session(package.session().id());
         } 
         // any other Z39.50 package is removed from statistics
         else {
@@ -222,22 +228,38 @@ void yf::LoadBalance::Impl::process(mp::Package &package)
             remove_package(package.session().id());
         }
     }
-}
-            
-// getting timestamp for receiving of package
-//boost::posix_time::ptime receive_time
-//    = boost::posix_time::microsec_clock::local_time();
-// //<< receive_time << " "
-// //<< to_iso_string(receive_time) << " "
-//<< to_iso_extended_string(receive_time) << " "
 
+    // finally removing sessions and marking deads
+    if (is_closed_back || is_closed_front){        
+        boost::mutex::scoped_lock scoped_lock(m_mutex);
+
+        // marking backend dead if backend closed without fronted close 
+        if (is_closed_front == false)
+            add_dead(package.session().id());
+
+        remove_session(package.session().id());
 
+        // making sure that package is closed
+        package.session().close();
+    }
+}
+            
 // statistic manipulating functions, 
 void yf::LoadBalance::Impl::add_dead(unsigned long session_id){
 
+    
     std::string target = find_session_target(session_id);
 
-    std::cout << "add_dead " << session_id << "\n";
+    if (target.size() != 0){
+        std::map<std::string, TargetStat>::iterator itarg;        
+        itarg = m_target_stat.find(target);
+        if (itarg != m_target_stat.end()
+            && itarg->second.deads < std::numeric_limits<unsigned int>::max()){
+            itarg->second.deads += 1;
+            // std:.cout << "add_dead " << session_id << " " << target 
+            //          << " d:" << itarg->second.deads << "\n";
+        }
+    }
 };
 
 //void yf::LoadBalance::Impl::clear_dead(unsigned long session_id){
@@ -251,10 +273,12 @@ void yf::LoadBalance::Impl::add_package(unsigned long session_id){
     if (target.size() != 0){
         std::map<std::string, TargetStat>::iterator itarg;        
         itarg = m_target_stat.find(target);
-        if (itarg != m_target_stat.end()){
+        if (itarg != m_target_stat.end()
+            && itarg->second.packages 
+               < std::numeric_limits<unsigned int>::max()){
             itarg->second.packages += 1;
-            std::cout << "add_package " << session_id << " " << target 
-                      << " p:" << itarg->second.packages << "\n";
+            // std:.cout << "add_package " << session_id << " " << target 
+            //          << " p:" << itarg->second.packages << "\n";
         }
     }
 };
@@ -265,10 +289,11 @@ void yf::LoadBalance::Impl::remove_package(unsigned long session_id){
     if (target.size() != 0){
         std::map<std::string, TargetStat>::iterator itarg;        
         itarg = m_target_stat.find(target);
-        if (itarg != m_target_stat.end()){
+        if (itarg != m_target_stat.end()
+            && itarg->second.packages > 0){
             itarg->second.packages -= 1;
-            std::cout << "remove_package " << session_id << " " << target 
-                      << " p:" << itarg->second.packages << "\n";
+            // std:.cout << "remove_package " << session_id << " " << target 
+            //          << " p:" << itarg->second.packages << "\n";
         }
     }
 };
@@ -289,16 +314,18 @@ void yf::LoadBalance::Impl::add_session(unsigned long session_id,
     if (itarg == m_target_stat.end()){
         TargetStat stat;
         stat.sessions = 1;
-        m_target_stat.insert(std::make_pair(target, stat));
-        std::cout << "add_session " << session_id << " " << target 
-                  << " s:1\n";
-    } else {
+        stat.packages = 0;  // no idea why the defaut constructor TargetStat()
+        stat.deads = 0;     // is not initializig this correctly to zero ??
+       m_target_stat.insert(std::make_pair(target, stat));
+        // std:.cout << "add_session " << session_id << " " << target 
+        //          << " s:1\n";
+    } 
+    else if (itarg->second.sessions < std::numeric_limits<unsigned int>::max())
+    {
         itarg->second.sessions += 1;
-        std::cout << "add_session " << session_id << " " << target 
-                  << " s:" << itarg->second.sessions << "\n";
+        // std:.cout << "add_session " << session_id << " " << target 
+        //          << " s:" << itarg->second.sessions << "\n";
     }
-    
-    
 };
 
 void yf::LoadBalance::Impl::remove_session(unsigned long session_id){
@@ -322,13 +349,14 @@ void yf::LoadBalance::Impl::remove_session(unsigned long session_id){
     }
     
     // counting session down
-    itarg->second.sessions -= 1;
+    if (itarg->second.sessions > 0)
+        itarg->second.sessions -= 1;
 
-    std::cout << "remove_session " << session_id << " " << target 
-              << " s:" << itarg->second.sessions << "\n";
+    // std:.cout << "remove_session " << session_id << " " << target 
+    //          << " s:" << itarg->second.sessions << "\n";
     
     // clearing empty sessions and targets
-    if (itarg->second.sessions == 0){
+    if (itarg->second.sessions == 0 && itarg->second.deads == 0 ){
         m_target_stat.erase(itarg);
         m_session_target.erase(isess);
     }
@@ -359,13 +387,24 @@ unsigned int yf::LoadBalance::Impl::cost(std::string target){
         }
     }
     
-    std::cout << "cost " << target << " c:" << cost << "\n";
+    //std::cout << "cost " << target << " c:" << cost << "\n";
     return cost;
 };
 
 unsigned int yf::LoadBalance::Impl::dead(std::string target){
-    std::cout << "dead " << target << "\n";
-    return 0;
+
+    unsigned int dead;
+
+    if (target.size() != 0){
+        std::map<std::string, TargetStat>::iterator itarg;        
+        itarg = m_target_stat.find(target);
+        if (itarg != m_target_stat.end()){
+            dead = itarg->second.deads;
+        }
+    }
+    
+    //std::cout << "dead " << target << " d:" << dead << "\n";
+    return dead;
 };
 
 
@@ -388,8 +427,9 @@ extern "C" {
 /*
  * Local variables:
  * c-basic-offset: 4
+ * c-file-style: "Stroustrup"
  * indent-tabs-mode: nil
- * c-file-style: "stroustrup"
  * End:
  * vim: shiftwidth=4 tabstop=8 expandtab
  */
+