Merge lp://staging/~linuxjedi/drizzle/trunk-drizzledump-bugs-part2 into lp://staging/drizzle/7.0
Proposed by
Andrew Hutchings
| Status: | Merged |
|---|---|
| Approved by: | Brian Aker |
| Approved revision: | 1981 |
| Merged at revision: | 1981 |
| Proposed branch: | lp://staging/~linuxjedi/drizzle/trunk-drizzledump-bugs-part2 |
| Merge into: | lp://staging/drizzle/7.0 |
| Diff against target: |
138 lines (+20/-21) 5 files modified
client/drizzledump.cc (+5/-6) client/drizzledump_data.cc (+5/-0) client/drizzledump_data.h (+1/-0) client/drizzledump_drizzle.cc (+2/-2) client/drizzledump_mysql.cc (+7/-13) |
| To merge this branch: | bzr merge lp://staging/~linuxjedi/drizzle/trunk-drizzledump-bugs-part2 |
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Lee Bieber (community) | Needs Fixing | ||
|
Review via email:
|
|||
Description of the change
General Drizzledump fixes:
* column comments were missing from dump files
* --databases option was broken
Migration fixes:
* FLOAT UNSIGNED without any precision and scale failed to migrate
* FLOAT with only precision and no scale lost the precision during migrate
To post a comment you must log in.
Revision history for this message
| Lee Bieber (kalebral-deactivatedaccount) wrote | # |
review:
Needs Fixing
Revision history for this message
| Andrew Hutchings (linuxjedi) wrote | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Getting merge conflicts, please re-merge with trunk
Text conflict in client/