Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-07-06 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60480/ --- (Updated July 6, 2017, 5:17 p.m.) Review request for Ambari, Dmytro Grinenko,

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-30 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60480/#review179354 --- Ship it! Ship It! - Jonathan Hurley On June 29, 2017, 11:47

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-29 Thread Di Li
> On June 28, 2017, 7:51 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/configuration/hive-env.xml > > Lines 118 (patched) > > > > > > This is missing > > Di

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-29 Thread Jonathan Hurley
> On June 28, 2017, 3:51 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/configuration/hive-env.xml > > Lines 118 (patched) > > > > > > This is missing > > Di

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-29 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60480/ --- (Updated June 29, 2017, 3:47 p.m.) Review request for Ambari, Dmytro Grinenko,

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-29 Thread Di Li
> On June 28, 2017, 7:51 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/configuration/hive-env.xml > > Lines 118 (patched) > > > > > > This is missing > > Di

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-28 Thread Di Li
> On June 28, 2017, 7:51 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/configuration/hive-env.xml > > Lines 118 (patched) > > > > > > This is missing right,

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-28 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60480/#review179157 ---

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-28 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60480/ --- (Updated June 28, 2017, 6:23 p.m.) Review request for Ambari, Dmytro Grinenko,

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-28 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60480/#review179142 ---

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-28 Thread Di Li
> On June 28, 2017, 2:48 p.m., Jonathan Hurley wrote: > > Should this also be put into trunk? We would still need to support this > > moving forward. Maybe we need a Jira to track an actual property which > > governs which database executable to invoke. > > Di Li wrote: > An updated

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-28 Thread Di Li
> On June 28, 2017, 2:48 p.m., Jonathan Hurley wrote: > > Should this also be put into trunk? We would still need to support this > > moving forward. Maybe we need a Jira to track an actual property which > > governs which database executable to invoke. An updated version should be pushed to

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-28 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60480/#review179109 --- Should this also be put into trunk? We would still need to

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-28 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60480/#review179108 --- Ship it! Ship It! - Jonathan Hurley On June 27, 2017, 4:08

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-27 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60480/#review179055 --- Ship it! Ship It! - Tim Thorpe On June 27, 2017, 8:08 p.m.,

Re: Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-27 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60480/#review179034 --- Ship it! Ship It! - Alejandro Fernandez On June 27, 2017,

Review Request 60480: AMBARI-21351 Add Support For MariaDB to Python Scripts

2017-06-27 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60480/ --- Review request for Ambari, Dmytro Grinenko, Di Li, Dmitro Lisnichenko, Jonathan