Last Comment Bug 577058 - --regenerate fails on PostgreSQL
: --regenerate fails on PostgreSQL
Product: Bugzilla
Classification: Server Software
Component: Reporting/Charting (show other bugs)
: 3.4
: All All
-- normal (vote)
: Bugzilla 3.6
Assigned To: Max Kanat-Alexander
: default-qa
Depends on:
  Show dependency treegraph
Reported: 2010-07-05 22:29 PDT by Max Kanat-Alexander
Modified: 2010-07-06 11:10 PDT (History)
1 user (show)
LpSolit: approval+
LpSolit: approval3.6+
mkanat: blocking3.6.2+
See Also:
QA Whiteboard:
Iteration: ---
Points: ---

v1 (460 bytes, patch)
2010-07-05 22:31 PDT, Max Kanat-Alexander
reed: review+
Details | Diff | Splinter Review

Description User image Max Kanat-Alexander 2010-07-05 22:29:45 PDT
Doing --regenerate fails on PostgreSQL (and probably any non-MySQL database) because there's a SQL error that is coincidentally worked around by the DB::Mysql implementation of sql_to_days.
Comment 1 User image Max Kanat-Alexander 2010-07-05 22:31:08 PDT
Created attachment 456140 [details] [diff] [review]

This adds a single space, to fix the problem. :-)
Comment 2 User image Reed Loden [:reed] (use needinfo?) 2010-07-05 23:02:24 PDT
Comment on attachment 456140 [details] [diff] [review]

r=me by inspection.

Surprised nobody noticed this sooner, considering it should throw some type of SQL error.
Comment 3 User image Max Kanat-Alexander 2010-07-06 11:10:02 PDT
Yeah, I'm surprised as well. Well, --regenerate isn't that commonly used or known about, I suppose.

In any case, thanks for the review! :-)

Committing to: bzr+ssh://
Committed revision 7283.

Committing to: bzr+ssh://
Committed revision 7122.

Note You need to log in before you can comment on or make changes to this bug.