Skip to content

Db optimization task 4#12

Open
MarynaNogtieva wants to merge 12 commits intospajic:masterfrom
MarynaNogtieva:db-optimization-task-4
Open

Db optimization task 4#12
MarynaNogtieva wants to merge 12 commits intospajic:masterfrom
MarynaNogtieva:db-optimization-task-4

Conversation

@MarynaNogtieva
Copy link

Result are in case study file metrics/case_study_english.md

Copy link
Owner

@spajic spajic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work!

@@ -1,4 +1,6 @@
Rails.application.routes.draw do
mount PgHero::Engine, at: "pghero"
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@@ -0,0 +1,7 @@
class AddIndiciesToTrips < ActiveRecord::Migration[5.2]
disable_ddl_transaction!
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 upvote for :concurrently

@@ -0,0 +1,6 @@
class AddForeignKeyToTrips < ActiveRecord::Migration[5.2]
def change
add_foreign_key :trips, :cities, column: :from_id, on_delete: :cascade
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants