You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Tom Beerbower <tb...@hortonworks.com> on 2014/04/08 00:28:11 UTC

Review Request 20108: Ambari Views : MySQL create script is wrong for View tables

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20108/
-----------------------------------------------------------

Review request for Ambari, John Speidel, Mahadev Konar, and Nate Cole.


Bugs: AMBARI-5385
    https://issues.apache.org/jira/browse/AMBARI-5385


Repository: ambari


Description
-------

The create table statements for views should not include 'ambari'...
{code}
CREATE TABLE ambari.viewmain (view_name VARCHAR(255) NOT NULL, label VARCHAR(255), version VARCHAR(255), archive VARCHAR(255), PRIMARY KEY(view_name));
{code}
Should be ...
{code}
CREATE TABLE viewmain (view_name VARCHAR(255) NOT NULL, label VARCHAR(255), version VARCHAR(255), archive VARCHAR(255), PRIMARY KEY(view_name));
{code}

Also, remove the GRANT ALL statements for the view tables ... 

{code}
GRANT ALL PRIVILEGES ON TABLE ambari.view TO :username;
{code} 


Diffs
-----

  ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 7db8b80 

Diff: https://reviews.apache.org/r/20108/diff/


Testing
-------

Results :

Tests run: 1519, Failures: 0, Errors: 0, Skipped: 9


------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 20:14.394s
[INFO] Finished at: Mon Apr 07 18:16:46 EDT 2014
[INFO] Final Memory: 36M/123M
[INFO] ------------------------------------------------------------------------


Thanks,

Tom Beerbower


Re: Review Request 20108: Ambari Views : MySQL create script is wrong for View tables

Posted by Tom Beerbower <tb...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20108/
-----------------------------------------------------------

(Updated April 7, 2014, 10:29 p.m.)


Review request for Ambari, John Speidel, Mahadev Konar, and Nate Cole.


Bugs: AMBARI-5385
    https://issues.apache.org/jira/browse/AMBARI-5385


Repository: ambari


Description
-------

The create table statements for views should not include 'ambari'...
{code}
CREATE TABLE ambari.viewmain (view_name VARCHAR(255) NOT NULL, label VARCHAR(255), version VARCHAR(255), archive VARCHAR(255), PRIMARY KEY(view_name));
{code}
Should be ...
{code}
CREATE TABLE viewmain (view_name VARCHAR(255) NOT NULL, label VARCHAR(255), version VARCHAR(255), archive VARCHAR(255), PRIMARY KEY(view_name));
{code}

Also, remove the GRANT ALL statements for the view tables ... 

{code}
GRANT ALL PRIVILEGES ON TABLE ambari.view TO :username;
{code} 


Diffs
-----

  ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 7db8b80 

Diff: https://reviews.apache.org/r/20108/diff/


Testing
-------

Results :

Tests run: 1519, Failures: 0, Errors: 0, Skipped: 9


------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 20:14.394s
[INFO] Finished at: Mon Apr 07 18:16:46 EDT 2014
[INFO] Final Memory: 36M/123M
[INFO] ------------------------------------------------------------------------


Thanks,

Tom Beerbower