From 3c62be695653772c3d47a7abc385f03834883aa9 Mon Sep 17 00:00:00 2001
From: Josh Cox <joshcox@uberatc.com>
Date: Tue, 23 Aug 2016 03:40:45 -0500
Subject: [PATCH] Add patch to remove conduit_connectionlog table (Fixes T9982)

Summary: Adds a schema patch that removes conduit_connectionlog. This table hasn't been used in 8ish months so it's probably safe to get rid of.

Test Plan: Apply the patch locally and confirm that the table does indeed get dropped.

Reviewers: #blessed_reviewers, epriestley

Reviewed By: #blessed_reviewers, epriestley

Subscribers: epriestley

Differential Revision: https://secure.phabricator.com/D16438
---
 .../autopatches/20160824.connectionlog.sql    |  1 +
 src/__phutil_library_map__.php                |  2 --
 .../PhabricatorConduitConnectionLog.php       | 26 -------------------
 3 files changed, 1 insertion(+), 28 deletions(-)
 create mode 100644 resources/sql/autopatches/20160824.connectionlog.sql
 delete mode 100644 src/applications/conduit/storage/PhabricatorConduitConnectionLog.php

diff --git a/resources/sql/autopatches/20160824.connectionlog.sql b/resources/sql/autopatches/20160824.connectionlog.sql
new file mode 100644
index 0000000000..499fc71845
--- /dev/null
+++ b/resources/sql/autopatches/20160824.connectionlog.sql
@@ -0,0 +1 @@
+DROP TABLE {$NAMESPACE}_conduit.conduit_connectionlog;
diff --git a/src/__phutil_library_map__.php b/src/__phutil_library_map__.php
index a6ab763246..e32c3d2c58 100644
--- a/src/__phutil_library_map__.php
+++ b/src/__phutil_library_map__.php
@@ -2114,7 +2114,6 @@ phutil_register_library_map(array(
     'PhabricatorConduitAPIController' => 'applications/conduit/controller/PhabricatorConduitAPIController.php',
     'PhabricatorConduitApplication' => 'applications/conduit/application/PhabricatorConduitApplication.php',
     'PhabricatorConduitCertificateToken' => 'applications/conduit/storage/PhabricatorConduitCertificateToken.php',
-    'PhabricatorConduitConnectionLog' => 'applications/conduit/storage/PhabricatorConduitConnectionLog.php',
     'PhabricatorConduitConsoleController' => 'applications/conduit/controller/PhabricatorConduitConsoleController.php',
     'PhabricatorConduitContentSource' => 'infrastructure/contentsource/PhabricatorConduitContentSource.php',
     'PhabricatorConduitController' => 'applications/conduit/controller/PhabricatorConduitController.php',
@@ -6846,7 +6845,6 @@ phutil_register_library_map(array(
     'PhabricatorConduitAPIController' => 'PhabricatorConduitController',
     'PhabricatorConduitApplication' => 'PhabricatorApplication',
     'PhabricatorConduitCertificateToken' => 'PhabricatorConduitDAO',
-    'PhabricatorConduitConnectionLog' => 'PhabricatorConduitDAO',
     'PhabricatorConduitConsoleController' => 'PhabricatorConduitController',
     'PhabricatorConduitContentSource' => 'PhabricatorContentSource',
     'PhabricatorConduitController' => 'PhabricatorController',
diff --git a/src/applications/conduit/storage/PhabricatorConduitConnectionLog.php b/src/applications/conduit/storage/PhabricatorConduitConnectionLog.php
deleted file mode 100644
index 498bb6ecfa..0000000000
--- a/src/applications/conduit/storage/PhabricatorConduitConnectionLog.php
+++ /dev/null
@@ -1,26 +0,0 @@
-<?php
-
-final class PhabricatorConduitConnectionLog extends PhabricatorConduitDAO {
-
-  protected $client;
-  protected $clientVersion;
-  protected $clientDescription;
-  protected $username;
-
-  protected function getConfiguration() {
-    return array(
-      self::CONFIG_COLUMN_SCHEMA => array(
-        'client' => 'text255?',
-        'clientVersion' => 'text255?',
-        'clientDescription' => 'text255?',
-        'username' => 'text255?',
-      ),
-      self::CONFIG_KEY_SCHEMA => array(
-        'key_created' => array(
-          'columns' => array('dateCreated'),
-        ),
-      ),
-    ) + parent::getConfiguration();
-  }
-
-}