----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57687/#review171587 -----------------------------------------------------------
Ship it! Ship It! - Alejandro Fernandez On April 9, 2017, 7:03 a.m., Masahiro Tanaka wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57687/ > ----------------------------------------------------------- > > (Updated April 9, 2017, 7:03 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Jayush Luniya, and Yusaku > Sako. > > > Bugs: AMBARI-20443 > https://issues.apache.org/jira/browse/AMBARI-20443 > > > Repository: ambari > > > Description > ------- > > In Advanced hive-env, there is a parameter named hive_ambari_database. Though > I change the parameter, it looks like nothing has changed. I greped the > entire ambari source code, the parameter is not referenced. > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/configuration/hive-env.xml > 5f56617 > > ambari-server/src/main/resources/common-services/HIVE/2.1.0.3.0/configuration/hive-env.xml > 4ed26f7 > > > Diff: https://reviews.apache.org/r/57687/diff/2/ > > > Testing > ------- > > -1 overall. Here are the results of testing the latest attachment > http://issues.apache.org/jira/secure/attachment/12858664/AMBARI-20443.0.patch > against trunk revision . > +1 @author. The patch does not contain any @author tags. > -1 tests included. The patch doesn't appear to include any new or modified > tests. > Please justify why no new tests are needed for this patch. > Also please list what manual steps were performed to verify this patch. > +1 javac. The applied patch does not increase the total number of javac > compiler warnings. > +1 release audit. The applied patch does not increase the total number of > release audit warnings. > +1 core tests. The patch passed unit tests in ambari-server. > Test results: > https://builds.apache.org/job/Ambari-trunk-test-patch/11023//testReport/ > Console output: > https://builds.apache.org/job/Ambari-trunk-test-patch/11023//console > This message is automatically generated. > > > Thanks, > > Masahiro Tanaka > >