[MCOL-317] replace drizzle with maria client library Created: 2016-09-23 Updated: 2017-03-02 Resolved: 2017-03-02 |
|
| Status: | Closed |
| Project: | MariaDB ColumnStore |
| Component/s: | ExeMgr |
| Affects Version/s: | 1.0.3 |
| Fix Version/s: | 1.1.0 |
| Type: | Bug | Priority: | Major |
| Reporter: | David Thompson (Inactive) | Assignee: | Daniel Lee (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Attachments: |
|
| Sprint: | 2017-01, 2017-2, 2017-3, 2017-4, 2017-5 |
| Description |
|
Usage of libdrizzle should be replaced by the client library since a much older version of drizzle is being used and is also not strictly necessary. |
| Comments |
| Comment by Andrew Hutchings (Inactive) [ 2017-01-06 ] |
|
Patch for this attached, need to apply this and remove the utils/mysqlcl_idb/ directory. We have no 1.1 tree yet so not in git yet. The patch also switches to unbuffered results for better performance and lower memory usage. |
| Comment by Andrew Hutchings (Inactive) [ 2017-01-12 ] |
|
Request review from Ben as this makes build changes. |
| Comment by Ben Thompson (Inactive) [ 2017-01-25 ] |
|
Reviewed, looks good. |
| Comment by Ben Thompson (Inactive) [ 2017-01-25 ] |
|
Out-of-tree build fails, as discussed. |
| Comment by Andrew Hutchings (Inactive) [ 2017-01-25 ] |
|
Fixed out-of-tree linking issue |
| Comment by Ben Thompson (Inactive) [ 2017-01-25 ] |
|
Reviewed, Builds |
| Comment by Andrew Hutchings (Inactive) [ 2017-01-26 ] |
|
Re-opened as it broke RPM build |
| Comment by Andrew Hutchings (Inactive) [ 2017-01-26 ] |
|
Review for RPM build fixes |
| Comment by Ben Thompson (Inactive) [ 2017-01-26 ] |
|
Reviewed |
| Comment by Daniel Lee (Inactive) [ 2017-03-02 ] |
|
Build tested: Github source [root@localhost columnstore]# cd mariadb-columnstore-server/ Merge pull request #31 from jbfavre/fix_deb_package_dependency [root@localhost mariadb-columnstore-server]# cd mariadb-columnstore-engine/ change the check for prompt back to the previous code Manually testing cross-engine join. |