X-Git-Url: https://git.tokkee.org/?a=blobdiff_plain;f=src%2Ffrontend%2Fquery.c;h=6015d7dd17275f27d01ef1f312321ca22d4975b9;hb=0b74a25a2fda7e77f20519642a2fc101fd46980b;hp=149750b05782dc80311278f495fc994ad1b9cf1f;hpb=740f1a8839faaf3033ea23f4cf2eea0dc0209d13;p=sysdb.git

diff --git a/src/frontend/query.c b/src/frontend/query.c
index 149750b..6015d7d 100644
--- a/src/frontend/query.c
+++ b/src/frontend/query.c
@@ -59,6 +59,43 @@ lookup_tojson(sdb_store_obj_t *obj, sdb_store_matcher_t *filter,
 	return sdb_store_json_emit_full(f, obj, filter);
 } /* lookup_tojson */
 
+static int
+query_exec(sdb_conn_t *conn, sdb_ast_node_t *ast)
+{
+	sdb_store_query_t *q;
+	sdb_strbuf_t *buf;
+	int status;
+
+	q = sdb_store_query_prepare(ast);
+	if (! q) {
+		/* this shouldn't happen */
+		sdb_strbuf_sprintf(conn->errbuf, "failed to compile AST");
+		sdb_log(SDB_LOG_ERR, "frontend: failed to compile AST");
+		return -1;
+	}
+
+	buf = sdb_strbuf_create(1024);
+	if (! buf) {
+		sdb_strbuf_sprintf(conn->errbuf, "Out of memory");
+		sdb_object_deref(SDB_OBJ(q));
+		return -1;
+	}
+	status = sdb_store_query_execute(q, buf, conn->errbuf);
+	if (status < 0) {
+		char query[conn->cmd_len + 1];
+		strncpy(query, sdb_strbuf_string(conn->buf), conn->cmd_len);
+		query[sizeof(query) - 1] = '\0';
+		sdb_log(SDB_LOG_ERR, "frontend: failed to execute query '%s'", query);
+	}
+	else
+		sdb_connection_send(conn, status,
+				(uint32_t)sdb_strbuf_len(buf), sdb_strbuf_string(buf));
+
+	sdb_strbuf_destroy(buf);
+	sdb_object_deref(SDB_OBJ(q));
+	return status < 0 ? status : 0;
+} /* query_exec */
+
 /*
  * public API
  */
@@ -68,9 +105,6 @@ sdb_fe_query(sdb_conn_t *conn)
 {
 	sdb_llist_t *parsetree;
 	sdb_ast_node_t *ast = NULL;
-
-	sdb_store_matcher_t *q;
-	sdb_strbuf_t *buf = NULL;
 	int status = 0;
 
 	if ((! conn) || (conn->cmd != SDB_CONNECTION_QUERY))
@@ -110,43 +144,12 @@ sdb_fe_query(sdb_conn_t *conn)
 			}
 	}
 
-	q = sdb_store_query_prepare(ast);
-	if (! q) {
-		/* this shouldn't happen */
-		sdb_strbuf_sprintf(conn->errbuf, "failed to compile AST");
-		sdb_log(SDB_LOG_ERR, "frontend: failed to compile AST");
-		status = -1;
-	} else {
-		buf = sdb_strbuf_create(1024);
-		if (! buf) {
-			sdb_strbuf_sprintf(conn->errbuf, "Out of memory");
-			sdb_object_deref(SDB_OBJ(q));
-			return -1;
-		}
-		status = sdb_store_query_execute(q, buf, conn->errbuf);
-		if (status < 0) {
-			char query[conn->cmd_len + 1];
-			strncpy(query, sdb_strbuf_string(conn->buf), conn->cmd_len);
-			query[sizeof(query) - 1] = '\0';
-			sdb_log(SDB_LOG_ERR, "frontend: failed to execute query '%s'", query);
-		}
+	if (ast) {
+		status = query_exec(conn, ast);
+		sdb_object_deref(SDB_OBJ(ast));
 	}
-	sdb_object_deref(SDB_OBJ(ast));
 	sdb_llist_destroy(parsetree);
-
-	if (status < 0) {
-		sdb_object_deref(SDB_OBJ(q));
-		sdb_strbuf_destroy(buf);
-		return status;
-	}
-
-	assert(buf);
-	sdb_connection_send(conn, status,
-			(uint32_t)sdb_strbuf_len(buf), sdb_strbuf_string(buf));
-
-	sdb_strbuf_destroy(buf);
-	sdb_object_deref(SDB_OBJ(q));
-	return 0;
+	return status;
 } /* sdb_fe_query */
 
 int
@@ -460,38 +463,5 @@ sdb_fe_exec_lookup(sdb_conn_t *conn, int type,
 	return 0;
 } /* sdb_fe_exec_lookup */
 
-int
-sdb_fe_exec_timeseries(sdb_conn_t *conn,
-		const char *hostname, const char *metric,
-		sdb_timeseries_opts_t *opts)
-{
-	sdb_strbuf_t *buf;
-	uint32_t res_type = htonl(SDB_CONNECTION_TIMESERIES);
-
-	buf = sdb_strbuf_create(1024);
-	if (! buf) {
-		char errbuf[1024];
-		sdb_log(SDB_LOG_ERR, "frontend: Failed to create "
-				"buffer to handle TIMESERIES command: %s",
-				sdb_strerror(errno, errbuf, sizeof(errbuf)));
-
-		sdb_strbuf_sprintf(conn->errbuf, "Out of memory");
-		return -1;
-	}
-
-	sdb_strbuf_memcpy(buf, &res_type, sizeof(uint32_t));
-	if (sdb_store_fetch_timeseries(hostname, metric, opts, buf)) {
-		sdb_log(SDB_LOG_ERR, "frontend: Failed to fetch time-series");
-		sdb_strbuf_sprintf(conn->errbuf, "Failed to fetch time-series");
-		sdb_strbuf_destroy(buf);
-		return -1;
-	}
-
-	sdb_connection_send(conn, SDB_CONNECTION_DATA,
-			(uint32_t)sdb_strbuf_len(buf), sdb_strbuf_string(buf));
-	sdb_strbuf_destroy(buf);
-	return 0;
-} /* sdb_fe_exec_timeseries */
-
 /* vim: set tw=78 sw=4 ts=4 noexpandtab : */