Merge lp://qastaging/~patrick-crews/randgen/bug961630 into lp://qastaging/randgen

Proposed by Patrick Crews
Status: Approved
Approved by: Philip Stoev
Approved revision: 743
Proposed branch: lp://qastaging/~patrick-crews/randgen/bug961630
Merge into: lp://qastaging/randgen
Diff against target: 21 lines (+2/-2)
1 file modified
lib/GenTest/App/Gendata.pm (+2/-2)
To merge this branch: bzr merge lp://qastaging/~patrick-crews/randgen/bug961630
Reviewer Review Type Date Requested Status
Philip Stoev Approve
Review via email: mp+98731@code.qastaging.launchpad.net

Description of the change

Updated GenData.pm to ensure we consistently quote table names during the main gendata operations.

To post a comment you must log in.
Revision history for this message
Philip Stoev (philip-stoev) wrote :

Ok to push. However, the backticks is non-standard -- double quotes are. So, if in the future new database products are added to Gendata.pm , this may need to change.

review: Approve

Unmerged revisions

743. By Patrick Crews

Update to Gendata.pm file to ensure we quote table names consistently

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.