Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

pg_namespace table isn't used. #8416

Merged
merged 1 commit into from

2 participants

Toshinori Kajihara Rafael Mendonça França
Toshinori Kajihara
Collaborator

When investigating #8414, I found a bit strange sql.
In this sql, pg_namespace (a.k.a. name) isn't joined other table.

If we want to use cross join, I'm wrong.

Rafael Mendonça França
Owner

@kennyj you are totally right. Thank you.

Rafael Mendonça França rafaelfranca merged commit 042f5eb into from
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Dec 4, 2012
  1. Toshinori Kajihara

    pg_namespace table isn't used.

    kennyj authored
This page is out of date. Refresh to see the latest.
1  activerecord/lib/active_record/connection_adapters/postgresql/schema_statements.rb
View
@@ -267,7 +267,6 @@ def pk_and_sequence_for(table) #:nodoc:
FROM pg_class seq,
pg_attribute attr,
pg_depend dep,
- pg_namespace name,
pg_constraint cons
WHERE seq.oid = dep.objid
AND seq.relkind = 'S'
Something went wrong with that request. Please try again.