Skip to content
This repository has been archived by the owner on Jun 23, 2022. It is now read-only.

feat(bulk_load): add query compaction status http interface #696

Merged
merged 15 commits into from
Dec 28, 2020
Merged
Show file tree
Hide file tree
Changes from 8 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
36 changes: 36 additions & 0 deletions src/replica/replica.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -443,6 +443,42 @@ std::string replica::query_compact_state() const
return _app->query_compact_state();
}

const char *manual_compaction_status_to_string(manual_compaction_status status)
{
switch (status) {
case FINISH:
return "CompactionFinish";
case RUNNING:
return "CompactionRunning";
case QUEUE:
return "CompactionQueue";
default:
dassert(false, "");
hycdong marked this conversation as resolved.
Show resolved Hide resolved
__builtin_unreachable();
}
}
foreverneverer marked this conversation as resolved.
Show resolved Hide resolved

manual_compaction_status replica::get_compact_status() const
{
std::string compact_state = query_compact_state();
// query_compact_state will return a message like:
// Case1. last finish at [-]
// - partition is not manual compaction
// Case2. last finish at [timestamp], last used {time_used} ms
// - partition manual compaction finished
// Case3. last finish at [-], recent enqueue at [timestamp]
// - partition is in manual compaction queue
// Case4. last finish at [-], recent enqueue at [timestamp], recent start at [timestamp]
// - partition is running manual compaction
if (compact_state.find("recent start at") != std::string::npos) {
hycdong marked this conversation as resolved.
Show resolved Hide resolved
return RUNNING;
} else if (compact_state.find("recent enqueue at") != std::string::npos) {
return QUEUE;
} else {
return FINISH;
}
}

// Replicas on the server which serves for the same table will share the same perf-counter.
// For example counter `table.level.RPC_RRDB_RRDB_MULTI_PUT.latency(ns)@test_table` is shared by
// all the replicas for `test_table`.
Expand Down
12 changes: 12 additions & 0 deletions src/replica/replica.h
Original file line number Diff line number Diff line change
Expand Up @@ -78,6 +78,14 @@ namespace test {
class test_checker;
}

enum manual_compaction_status
{
FINISH = 0,
RUNNING,
QUEUE
};
const char *manual_compaction_status_to_string(manual_compaction_status status);

class replica : public serverlet<replica>, public ref_counter, public replica_base
{
public:
Expand Down Expand Up @@ -392,8 +400,12 @@ class replica : public serverlet<replica>, public ref_counter, public replica_ba

void update_restore_progress(uint64_t f_size);

// Used for remote command
hycdong marked this conversation as resolved.
Show resolved Hide resolved
std::string query_compact_state() const;

// Used for http interface
manual_compaction_status get_compact_status() const;

void init_table_level_latency_counters();

void on_detect_hotkey(const detect_hotkey_request &req, /*out*/ detect_hotkey_response &resp);
Expand Down
42 changes: 42 additions & 0 deletions src/replica/replica_http_service.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@

#include <nlohmann/json.hpp>
#include <fmt/format.h>
#include <dsn/utility/output_utils.h>
#include "replica_http_service.h"
#include "duplication/duplication_sync_timer.h"

Expand Down Expand Up @@ -55,5 +56,46 @@ void replica_http_service::query_duplication_handler(const http_request &req, ht
resp.body = json.dump();
}

void replica_http_service::query_compaction_handler(const http_request &req, http_response &resp)
{
auto it = req.query_args.find("app_id");
if (it == req.query_args.end()) {
resp.body = "app_id should not be empty";
resp.status_code = http_status_code::bad_request;
return;
}

int32_t app_id = -1;
if (!buf2int32(it->second, app_id) || app_id < 0) {
resp.body = fmt::format("invalid app_id={}", it->second);
resp.status_code = http_status_code::bad_request;
return;
}

std::unordered_map<gpid, manual_compaction_status> partition_compaction_status;
_stub->query_app_compact_status(app_id, partition_compaction_status);

int32_t running_count = 0;
int32_t queue_count = 0;
int32_t finish_count = 0;
for (const auto &kv : partition_compaction_status) {
if (kv.second == RUNNING) {
running_count++;
} else if (kv.second == QUEUE) {
queue_count++;
} else if (kv.second == FINISH) {
finish_count++;
}
}
dsn::utils::table_printer tp("status");
tp.add_row_name_and_data(manual_compaction_status_to_string(RUNNING), running_count);
tp.add_row_name_and_data(manual_compaction_status_to_string(QUEUE), queue_count);
tp.add_row_name_and_data(manual_compaction_status_to_string(FINISH), finish_count);
std::ostringstream out;
tp.output(out, dsn::utils::table_printer::output_format::kJsonCompact);
resp.body = out.str();
resp.status_code = http_status_code::ok;
}

} // namespace replication
} // namespace dsn
7 changes: 7 additions & 0 deletions src/replica/replica_http_service.h
Original file line number Diff line number Diff line change
Expand Up @@ -20,11 +20,18 @@ class replica_http_service : public http_service
std::placeholders::_1,
std::placeholders::_2),
"ip:port/replica/duplication?appid=<appid>");
register_handler("compaction",
std::bind(&replica_http_service::query_compaction_handler,
this,
std::placeholders::_1,
std::placeholders::_2),
"ip:port/replica/compaction?app_id=<app_id>");
}

std::string path() const override { return "replica"; }

void query_duplication_handler(const http_request &req, http_response &resp);
void query_compaction_handler(const http_request &req, http_response &resp);

private:
replica_stub *_stub;
Expand Down
12 changes: 12 additions & 0 deletions src/replica/replica_stub.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2807,5 +2807,17 @@ void replica_stub::on_detect_hotkey(detect_hotkey_rpc rpc)
response.err_hint = fmt::format("not find the replica {} \n", request.pid);
}
}

void replica_stub::query_app_compact_status(
int32_t app_id, std::unordered_map<gpid, manual_compaction_status> &status)
{
zauto_read_lock l(_replicas_lock);
for (auto it = _replicas.begin(); it != _replicas.end(); ++it) {
if (it->first.get_app_id() == app_id) {
status[it->first] = it->second->get_compact_status();
}
}
}

} // namespace replication
} // namespace dsn
5 changes: 5 additions & 0 deletions src/replica/replica_stub.h
Original file line number Diff line number Diff line change
Expand Up @@ -217,6 +217,11 @@ class replica_stub : public serverlet<replica_stub>, public ref_counter
void on_query_disk_info(query_disk_info_rpc rpc);
void on_disk_migrate(replica_disk_migrate_rpc rpc);

// query partitions compact status by app_id
void
query_app_compact_status(int32_t app_id,
/*out*/ std::unordered_map<gpid, manual_compaction_status> &status);

private:
enum replica_node_state
{
Expand Down
35 changes: 35 additions & 0 deletions src/replica/test/replica_test.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
#include <dsn/utility/fail_point.h>
#include "replica_test_base.h"
#include <dsn/utility/defer.h>
#include "replica/replica_http_service.h"

namespace dsn {
namespace replication {
Expand All @@ -21,6 +22,7 @@ class replica_test : public replica_test_base
public:
void SetUp() override
{
FLAGS_enable_http_server = false;
stub->install_perf_counters();
mock_app_info();
_mock_replica = stub->generate_replica(_app_info, pid, partition_status::PS_PRIMARY, 1);
Expand Down Expand Up @@ -81,5 +83,38 @@ TEST_F(replica_test, backup_request_qps)
ASSERT_GT(get_table_level_backup_request_qps(), 0);
}

TEST_F(replica_test, query_compaction_test)
{
replica_http_service http_svc(stub.get());
struct query_compaction_test
{
std::string app_id;
http_status_code expected_code;
std::string expected_response_json;
} tests[] = {
{"", http_status_code::bad_request, "app_id should not be empty"},
{"xxx", http_status_code::bad_request, "invalid app_id=xxx"},
{"2",
http_status_code::ok,
R"({"status":{"CompactionRunning":"0","CompactionQueue":"0","CompactionFinish":"1"}})"},
{"4",
http_status_code::ok,
R"({"status":{"CompactionRunning":"0","CompactionQueue":"0","CompactionFinish":"0"}})"}};
for (const auto &test : tests) {
http_request req;
http_response resp;
if (!test.app_id.empty()) {
req.query_args["app_id"] = test.app_id;
}
http_svc.query_compaction_handler(req, resp);
ASSERT_EQ(resp.status_code, test.expected_code);
std::string expected_json = test.expected_response_json;
if (test.expected_code == http_status_code::ok) {
expected_json += "\n";
}
ASSERT_EQ(resp.body, expected_json);
}
}

} // namespace replication
} // namespace dsn