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

Bug795223 use postgres 9.2 for dev #849

Merged
merged 1 commit into from Dec 5, 2012

Conversation

rhelmer
Copy link
Contributor

@rhelmer rhelmer commented Sep 28, 2012

No description provided.

@selenamarie
Copy link
Contributor

Found a few issues:

  • Missing tables:
    daily_crash_codes
    productdims
  • Missing data for rapid_beta_version in products.csv (needs to be \d+.\d for DOMAIN)
  • Problem in backfill:
    INFO: backfilling reports_clean
    INFO: backfilling 06:00:00 of reports_clean starting at 2012-06-15 00:00:00+00
    CONTEXT: SQL statement "SELECT backfill_reports_clean( first_rc, last_rc )"
    PL/pgSQL function backfill_matviews(date,date,boolean) line 49 at PERFORM
    ERROR: invalid value for parameter "temp_buffers": 65536
    DETAIL: "temp_buffers" cannot be changed after any temporary tables have been accessed in the session.
    CONTEXT: SQL statement "SELECT update_reports_clean( cur_time, cyclesize, false )"
    PL/pgSQL function backfill_reports_clean(timestamp with time zone,timestamp with time zone) line 38 at PERFORM
    SQL statement "SELECT backfill_reports_clean( first_rc, last_rc )"
    PL/pgSQL function backfill_matviews(date,date,boolean) line 49 at PERFORM
    UPDATE 0

Haven't gotten to the bottom of that one yet, so there may be more issues beyond it.

@jberkus
Copy link
Contributor

jberkus commented Oct 1, 2012

daily_crash_codes and productdims are obsolete tables which were purged in the Mobeta deployment. Any code currently referring to them needs to be altered or removed.

@lonnen
Copy link
Contributor

lonnen commented Oct 19, 2012

Needs rebasing.

@@ -139,7 +139,7 @@ def main(self):

with PostgreSQLManager(dsn, self.config.logger) as db:
db_version = db.version()
if not re.match(r'9\.[01][.*]', db_version):
if not re.match(r'9\.[012][.*]', db_version):
print 'ERROR - unrecognized PostgreSQL vesion: %s' % db_version
print 'Only 9.0.x and 9.1.x are supported at this time.'
return 1
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we should leave all the changes in setupdb_app.py out of this particular patch. I made changes that restrict support to only 9.2. I haven't tested backward compatibility with 9.0 and our new schema dumps.

lauraxt added a commit that referenced this pull request Dec 5, 2012
Bug795223 use postgres 9.2 for dev
@lauraxt lauraxt merged commit a229a53 into mozilla-services:master Dec 5, 2012
rhelmer pushed a commit to rhelmer/socorro that referenced this pull request Jul 30, 2013
…ers-stuff

Faster query and better logging for deduplicate_users command.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants