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

GCC Puller Primary Key #910

Closed
wants to merge 2 commits into from
Closed

GCC Puller Primary Key #910

wants to merge 2 commits into from

Conversation

MelinaGoula
Copy link

What this pull request accomplishes:

  • modify create_audited_table() to only create a primary key when a new table is created
  • fixes the issue of creating multiple primary keys for a table

Issue(s) this solves:

What, in particular, needs to reviewed:

  • gcc_puller_functions.py

What needs to be done by a sysadmin after this PR is merged

  • testing of the new code

@MelinaGoula MelinaGoula self-assigned this Mar 27, 2024
@MelinaGoula MelinaGoula linked an issue Mar 27, 2024 that may be closed by this pull request
@MelinaGoula MelinaGoula changed the title 900 gcc puller pk GCC Puller Primary Key Mar 27, 2024
@gabrielwol gabrielwol added the gis label Jun 27, 2024
@chmnata
Copy link
Collaborator

chmnata commented Jul 10, 2024

Thanks a lot for the update, upon further investigation, the issue seems to be having an unrenamed table in the database, e.g. _loop_detector from a previous failed action. So the duplicated primary key error was alerting about adding the same primary key to the previously unrenamed tabled. This issue was resolved after deleting the unrenamed table in the database.

@chmnata chmnata closed this Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GCC Puller: Create primary key only if not exist
3 participants