You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by dm...@apache.org on 2018/11/22 09:48:07 UTC

[ambari] 02/02: AMBARI-24936. Fix suspicious code spotted while working on AMBARI-24936 - address review comments (dlysnichenko)

This is an automated email from the ASF dual-hosted git repository.

dmitriusan pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ambari.git

commit c36b478ab31ef6c6d9ac1194b1fe5c6a231f50ea
Author: Lisnichenko Dmitro <dl...@hortonworks.com>
AuthorDate: Wed Nov 21 21:31:53 2018 +0200

    AMBARI-24936. Fix suspicious code spotted while working on AMBARI-24936 - address review comments (dlysnichenko)
---
 .../test/java/org/apache/ambari/server/orm/DBAccessorImplTest.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/ambari-server/src/test/java/org/apache/ambari/server/orm/DBAccessorImplTest.java b/ambari-server/src/test/java/org/apache/ambari/server/orm/DBAccessorImplTest.java
index 99d463b..f591156 100644
--- a/ambari-server/src/test/java/org/apache/ambari/server/orm/DBAccessorImplTest.java
+++ b/ambari-server/src/test/java/org/apache/ambari/server/orm/DBAccessorImplTest.java
@@ -830,8 +830,8 @@ public class DBAccessorImplTest {
     assertEquals(String.format("SELECT id, name FROM %s WHERE time='100'", tableName),
     dbAccessor.buildQuery(tableName, new String[] {"id", "name"}, new String[] {"time"}, new String[] {"100"}));
 
-    assertEquals(String.format("SELECT id, name FROM %s", tableName),
-    dbAccessor.buildQuery(tableName, new String[] {"id", "name"}, null, null));
+    assertEquals(String.format("SELECT id, name, time FROM %s", tableName),
+    dbAccessor.buildQuery(tableName, new String[] {"id", "name", "time"}, null, null));
 
     try {
       dbAccessor.buildQuery("invalid_table_name", new String[] {"id", "name"}, new String[] {"time"}, new String[] {"100"});