Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Problems setting a custom primary key in a Rails 4 migration

I use postgresql 9.3, Ruby 2.0, Rails 4.0.0.

After reading numerous questions on SO regarding setting the Primary key on a table, I generated and added the following migration:

class CreateShareholders < ActiveRecord::Migration
  def change
    create_table :shareholders, { id: false, primary_key: :uid  } do |t|
      t.integer :uid, limit: 8
      t.string :name
      t.integer :shares

      t.timestamps
    end
  end
end

I also added self.primary_key = "uid" to my model.

The migration runs successfully, but when I connect to the DB using pgAdmin III I see that the uid column is not set as primary key. What am I missing?

like image 386
Alexander Popov Avatar asked Sep 27 '13 12:09

Alexander Popov


2 Answers

Take a look at this answer. Try to execute "ALTER TABLE shareholders ADD PRIMARY KEY (uid);" without specifying primary_key parameter in create_table block.

I suggest to write your migration like this (so you could rollback normally):

class CreateShareholders < ActiveRecord::Migration
  def up
    create_table :shareholders, id: false do |t|
      t.integer :uid, limit: 8
      t.string :name
      t.integer :shares

      t.timestamps
    end
    execute "ALTER TABLE shareholders ADD PRIMARY KEY (uid);"
  end

  def down
    drop_table :shareholders
  end
end

UPD: There is natural way (found here), but only with int4 type:

class CreateShareholders < ActiveRecord::Migration
  def change
    create_table :shareholders, id: false do |t|
      t.primary_key :uid
      t.string :name
      t.integer :shares

      t.timestamps
    end    
  end
end
like image 97
peresleguine Avatar answered Nov 06 '22 23:11

peresleguine


In my environment(activerecord 3.2.19 and postgres 9.3.1),

:id => true, :primary_key => "columname"

creates a primary key successfully but instead of specifying ":limit => 8" the column' type is int4!

create_table :m_check_pattern, :primary_key => "checkpatternid" do |t|
  t.integer     :checkpatternid, :limit => 8, :null => false
end

Sorry for the incomplete info.

like image 24
kairya1975 Avatar answered Nov 07 '22 01:11

kairya1975