ylqjgm
/
canvas-lms
mirror of https://github.com/instructure/canvas-lms.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Revert "add index on content_tags to make lookup / bulk updating easier" 

This reverts commit dbaf24a057.

Reason for revert: some urls are stupidly long and un-indexable

Change-Id: I9dafdfe5591bb811a8faead824a2d177eb71d604
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/312270
Reviewed-by: Ryan Norton <rnorton@instructure.com>
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
QA-Review: Jacob Burroughs <jburroughs@instructure.com>
Product-Review: Jacob Burroughs <jburroughs@instructure.com>
Migration-Review: Jacob Burroughs <jburroughs@instructure.com>
This commit is contained in:
Jacob Burroughs 2023-03-02 17:13:38 +00:00
parent cd012fa960
commit ac948b6aff
1 changed files with 0 additions and 28 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
db/migrate/20230219072741_index_content_tag.rb
View File

@ -1,28 +0,0 @@
# frozen_string_literal: true
#
# Copyright (C) 2023 - present Instructure, Inc.
#
# This file is part of Canvas.
#
# Canvas is free software: you can redistribute it and/or modify it under
# the terms of the GNU Affero General Public License as published by the Free
# Software Foundation, version 3 of the License.
#
# Canvas 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 Affero General Public License for more
# details.
#
# You should have received a copy of the GNU Affero General Public License along
# with this program. If not, see <http://www.gnu.org/licenses/>.
#
class IndexContentTag < ActiveRecord::Migration[6.1]
tag :postdeploy
disable_ddl_transaction!
def change
add_index :content_tags, %i[content_id content_type url], algorithm: :concurrently, if_not_exists: true
end
end