Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#1234 On OracleDB, a "Table has no primary key" error is raised when the index and the constraint name don't match #889

Closed
wants to merge 1 commit into from

Conversation

nitso
Copy link

@nitso nitso commented Jul 30, 2015

When reverse-engineering oracle database, "Table has no primary key" error is triggered. That is caused by a mistake in index selection from database.

When reverse-engineering oracle database, "Table has no primary key" error is triggered. That is caused by a mistake in index selection from database.
@deeky666
Copy link
Member

@nitso this needs a failing test case (functional).

@deeky666
Copy link
Member

deeky666 commented Jan 9, 2016

@nitso ping. We need at least information about how to reproduce the error. @DeepDiver1975 maybe you have a clue?

@athos99
Copy link

athos99 commented Sep 27, 2016

I'have the same probleme, for reproduce this bug, in the constraint dfintion, the CONSTRAINT_NAME must be different from the INDEX_NAME.

The 'nitso' solution is ok for me

@Ocramius
Copy link
Member

Handled in #2686

@Ocramius Ocramius closed this May 20, 2017
@Ocramius Ocramius self-assigned this May 20, 2017
@Ocramius Ocramius changed the title Fix "Table has no primary key" error for Oracle platform On OracleDB, a "Table has no primary key" error is raised when the index and the constraint name don't match Jul 22, 2017
@Ocramius Ocramius changed the title On OracleDB, a "Table has no primary key" error is raised when the index and the constraint name don't match #1234 On OracleDB, a "Table has no primary key" error is raised when the index and the constraint name don't match Jul 22, 2017
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants