From a3be8073e242db6bca4a94c2dccc55fa5a44a541 Mon Sep 17 00:00:00 2001
From: magenoxx <magenoxx@gmail.com>
Date: Sun, 10 Mar 2013 15:57:30 +0400
Subject: [PATCH] Moved job schedule time value to constant

---
 .../src/main/java/mage/server/TableManager.java        | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/Mage.Server/src/main/java/mage/server/TableManager.java b/Mage.Server/src/main/java/mage/server/TableManager.java
index 3a533eeb5d7..167334946f5 100644
--- a/Mage.Server/src/main/java/mage/server/TableManager.java
+++ b/Mage.Server/src/main/java/mage/server/TableManager.java
@@ -62,6 +62,13 @@ public class TableManager {
     private ConcurrentHashMap<UUID, TableController> controllers = new ConcurrentHashMap<UUID, TableController>();
     private ConcurrentHashMap<UUID, Table> tables = new ConcurrentHashMap<UUID, Table>();
 
+    /**
+     * Defines how often checking process should be run on server.
+     *
+     * In minutes.
+     */
+    private static final int EXPIRE_CHECK_PERIOD = 10;
+
     /**
      * This parameters defines when table can be counted as expired.
      * Uses EXPIRE_TIME_UNIT_VALUE as unit of measurement.
@@ -86,7 +93,7 @@ public class TableManager {
             public void run() {
                 checkExpired();
             }
-        }, 10, 10, TimeUnit.MINUTES);
+        }, EXPIRE_CHECK_PERIOD, EXPIRE_CHECK_PERIOD, TimeUnit.MINUTES);
     }
 
     public Table createTable(UUID roomId, UUID userId, MatchOptions options) {
@@ -268,7 +275,6 @@ public class TableManager {
         List<UUID> toRemove = new ArrayList<UUID>();
         for (Table table : tables.values()) {
             long diff = (now.getTime() - table.getCreateTime().getTime()) / EXPIRE_TIME_UNIT_VALUE;
-            logger.info("Expire = " + diff);
             if (diff >= EXPIRE_TIME) {
                 logger.info("Table expired: id = " + table.getId() + ", created_by=" + table.getControllerName() + ". Removing...");
                 toRemove.add(table.getId());