Skip to content

Commit da945b8

Browse files
committed
added clusterAllReplicas for reading query_log
1 parent 3deeb33 commit da945b8

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

client-v2/src/test/java/com/clickhouse/client/HttpTransportTests.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -806,7 +806,7 @@ public void testUserAgentHasCompleteProductName(String clientName, Pattern userA
806806
client.execute("SYSTEM FLUSH LOGS").get().close();
807807

808808
List<GenericRecord> logRecords = client.queryAll("SELECT http_user_agent, http_referer, " +
809-
" forwarded_for FROM system.query_log WHERE query_id = '" + q1Id + "'");
809+
" forwarded_for FROM clusterAllReplicas('default', system.query_log) WHERE query_id = '" + q1Id + "'");
810810
Assert.assertFalse(logRecords.isEmpty(), "No records found in query log");
811811

812812
for (GenericRecord record : logRecords) {
@@ -864,7 +864,7 @@ public void testClientName(String clientName, boolean setWithUpdate, String user
864864
client.execute("SYSTEM FLUSH LOGS").get().close();
865865

866866
List<GenericRecord> logRecords = client.queryAll("SELECT query_id, client_name, http_user_agent, http_referer " +
867-
" FROM system.query_log WHERE query_id = '" + settings.getQueryId() + "'");
867+
" FROM clusterAllReplicas('default', system.query_log) WHERE query_id = '" + settings.getQueryId() + "'");
868868
Assert.assertEquals(logRecords.get(0).getString("query_id"), settings.getQueryId());
869869
final String logUserAgent = logRecords.get(0).getString("http_user_agent");
870870
Assert.assertTrue(logUserAgent.startsWith(expectedClientNameStartsWith),
@@ -905,7 +905,7 @@ public void testClientNameThruRawOptions(String property, String value, boolean
905905
client.execute("SYSTEM FLUSH LOGS").get().close();
906906

907907
List<GenericRecord> logRecords = client.queryAll("SELECT query_id, client_name, http_user_agent, http_referer " +
908-
" FROM system.query_log WHERE query_id = '" + settings.getQueryId() + "'");
908+
" FROM clusterAllReplicas('default', system.query_log) WHERE query_id = '" + settings.getQueryId() + "'");
909909
Assert.assertEquals(logRecords.get(0).getString("query_id"), settings.getQueryId());
910910
final String logUserAgent = logRecords.get(0).getString("http_user_agent");
911911
Assert.assertTrue(logUserAgent.startsWith(value),

jdbc-v2/src/test/java/com/clickhouse/jdbc/ConnectionTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -311,7 +311,7 @@ public void setAndGetClientInfoTest(String clientName) throws SQLException {
311311
stmt.execute("SYSTEM FLUSH LOGS");
312312

313313

314-
final String logQuery ="SELECT http_user_agent FROM system.query_log WHERE query_id = " + stmt.enquoteLiteral(queryId);
314+
final String logQuery ="SELECT http_user_agent FROM clusterAllReplicas('default', system.query_log) WHERE query_id = " + stmt.enquoteLiteral(queryId);
315315
try (ResultSet rs = stmt.executeQuery(logQuery)) {
316316
Assert.assertTrue(rs.next());
317317
String userAgent = rs.getString("http_user_agent");
@@ -346,7 +346,7 @@ private void influenceUserAgentTest(String clientName, String urlParam) throws S
346346
stmt.execute("SYSTEM FLUSH LOGS");
347347

348348
final String logQuery ="SELECT http_user_agent " +
349-
" FROM system.query_log WHERE query = '" + testQuery.replaceAll("'", "\\\\'") + "'";
349+
" FROM clusterAllReplicas('default', system.query_log) WHERE query = '" + testQuery.replaceAll("'", "\\\\'") + "'";
350350
try (ResultSet rs = stmt.executeQuery(logQuery)) {
351351
Assert.assertTrue(rs.next());
352352
String userAgent = rs.getString("http_user_agent");

0 commit comments

Comments
 (0)