Skip to content

Commit ebebb6a

Browse files
committed
refactor(native): Remove folly::StringPiece in Counters.h
1 parent 1bd47cb commit ebebb6a

File tree

1 file changed

+65
-65
lines changed
  • presto-native-execution/presto_cpp/main/common

1 file changed

+65
-65
lines changed

presto-native-execution/presto_cpp/main/common/Counters.h

Lines changed: 65 additions & 65 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
*/
1414
#pragma once
1515

16-
#include <folly/Range.h>
16+
#include <string_view>
1717

1818
// Here we have all the counters presto cpp worker would export.
1919
namespace facebook::presto {
@@ -22,120 +22,120 @@ namespace facebook::presto {
2222
// See velox/common/base/StatsReporter.h for the interface.
2323
void registerPrestoMetrics();
2424

25-
constexpr folly::StringPiece kCounterDriverCPUExecutorQueueSize{
25+
constexpr std::string_view kCounterDriverCPUExecutorQueueSize{
2626
"presto_cpp.driver_cpu_executor_queue_size"};
27-
constexpr folly::StringPiece kCounterDriverCPUExecutorLatencyMs{
27+
constexpr std::string_view kCounterDriverCPUExecutorLatencyMs{
2828
"presto_cpp.driver_cpu_executor_latency_ms"};
29-
constexpr folly::StringPiece kCounterSpillerExecutorQueueSize{
29+
constexpr std::string_view kCounterSpillerExecutorQueueSize{
3030
"presto_cpp.spiller_executor_queue_size"};
31-
constexpr folly::StringPiece kCounterSpillerExecutorLatencyMs{
31+
constexpr std::string_view kCounterSpillerExecutorLatencyMs{
3232
"presto_cpp.spiller_executor_latency_ms"};
33-
constexpr folly::StringPiece kCounterHTTPExecutorLatencyMs{
33+
constexpr std::string_view kCounterHTTPExecutorLatencyMs{
3434
"presto_cpp.http_executor_latency_ms"};
35-
constexpr folly::StringPiece kCounterNumHTTPRequest{
35+
constexpr std::string_view kCounterNumHTTPRequest{
3636
"presto_cpp.num_http_request"};
37-
constexpr folly::StringPiece kCounterNumHTTPRequestError{
37+
constexpr std::string_view kCounterNumHTTPRequestError{
3838
"presto_cpp.num_http_request_error"};
39-
constexpr folly::StringPiece kCounterHTTPRequestLatencyMs{
39+
constexpr std::string_view kCounterHTTPRequestLatencyMs{
4040
"presto_cpp.http_request_latency_ms"};
4141
constexpr folly::StringPiece kCounterHTTPRequestSizeBytes{
4242
"presto_cpp.http_request_size_bytes"};
4343

44-
constexpr folly::StringPiece kCounterHttpClientNumConnectionsCreated{
44+
constexpr std::string_view kCounterHttpClientNumConnectionsCreated{
4545
"presto_cpp.http.client.num_connections_created"};
46-
constexpr folly::StringPiece kCounterHTTPClientTransactionCreateDelay{
46+
constexpr std::string_view kCounterHTTPClientTransactionCreateDelay{
4747
"presto_cpp.http.client.transaction_create_delay_ms"};
4848
/// Peak number of bytes queued in PrestoExchangeSource waiting for consume.
49-
constexpr folly::StringPiece kCounterExchangeSourcePeakQueuedBytes{
49+
constexpr std::string_view kCounterExchangeSourcePeakQueuedBytes{
5050
"presto_cpp.exchange_source_peak_queued_bytes"};
5151

52-
constexpr folly::StringPiece kCounterExchangeRequestDuration{
52+
constexpr std::string_view kCounterExchangeRequestDuration{
5353
"presto_cpp.exchange.request.duration"};
54-
constexpr folly::StringPiece kCounterExchangeRequestNumTries{
54+
constexpr std::string_view kCounterExchangeRequestNumTries{
5555
"presto_cpp.exchange.request.num_tries"};
56-
constexpr folly::StringPiece kCounterExchangeRequestPageSize{
56+
constexpr std::string_view kCounterExchangeRequestPageSize{
5757
"presto_cpp.exchange.request.page_size"};
5858

59-
constexpr folly::StringPiece kCounterExchangeGetDataSizeDuration{
59+
constexpr std::string_view kCounterExchangeGetDataSizeDuration{
6060
"presto_cpp.exchange.get_data_size.duration"};
61-
constexpr folly::StringPiece kCounterExchangeGetDataSizeNumTries{
61+
constexpr std::string_view kCounterExchangeGetDataSizeNumTries{
6262
"presto_cpp.exchange.get_data_size.num_tries"};
6363

64-
constexpr folly::StringPiece kCounterNumQueryContexts{
64+
constexpr std::string_view kCounterNumQueryContexts{
6565
"presto_cpp.num_query_contexts"};
6666
/// Export total bytes used by memory manager (in queries' memory pools).
67-
constexpr folly::StringPiece kCounterMemoryManagerTotalBytes{
67+
constexpr std::string_view kCounterMemoryManagerTotalBytes{
6868
"presto_cpp.memory_manager_total_bytes"};
6969

70-
constexpr folly::StringPiece kCounterNumTasks{"presto_cpp.num_tasks"};
71-
constexpr folly::StringPiece kCounterNumTasksBytesProcessed{
70+
constexpr std::string_view kCounterNumTasks{"presto_cpp.num_tasks"};
71+
constexpr std::string_view kCounterNumTasksBytesProcessed{
7272
"presto_cpp.num_tasks_bytes_processed"};
73-
constexpr folly::StringPiece kCounterNumTasksRunning{
73+
constexpr std::string_view kCounterNumTasksRunning{
7474
"presto_cpp.num_tasks_running"};
75-
constexpr folly::StringPiece kCounterNumTasksFinished{
75+
constexpr std::string_view kCounterNumTasksFinished{
7676
"presto_cpp.num_tasks_finished"};
77-
constexpr folly::StringPiece kCounterNumTasksCancelled{
77+
constexpr std::string_view kCounterNumTasksCancelled{
7878
"presto_cpp.num_tasks_cancelled"};
79-
constexpr folly::StringPiece kCounterNumTasksAborted{
79+
constexpr std::string_view kCounterNumTasksAborted{
8080
"presto_cpp.num_tasks_aborted"};
81-
constexpr folly::StringPiece kCounterNumTasksFailed{
81+
constexpr std::string_view kCounterNumTasksFailed{
8282
"presto_cpp.num_tasks_failed"};
8383
/// Number of the created but not yet started tasks, including queued tasks.
84-
constexpr folly::StringPiece kCounterNumTasksPlanned{
84+
constexpr std::string_view kCounterNumTasksPlanned{
8585
"presto_cpp.num_tasks_planned"};
8686
/// Number of the created tasks in the task queue.
87-
constexpr folly::StringPiece kCounterNumTasksQueued{
87+
constexpr std::string_view kCounterNumTasksQueued{
8888
"presto_cpp.num_tasks_queued"};
8989

90-
constexpr folly::StringPiece kCounterNumZombieVeloxTasks{
90+
constexpr std::string_view kCounterNumZombieVeloxTasks{
9191
"presto_cpp.num_zombie_velox_tasks"};
92-
constexpr folly::StringPiece kCounterNumZombiePrestoTasks{
92+
constexpr std::string_view kCounterNumZombiePrestoTasks{
9393
"presto_cpp.num_zombie_presto_tasks"};
94-
constexpr folly::StringPiece kCounterNumTasksWithStuckOperator{
94+
constexpr std::string_view kCounterNumTasksWithStuckOperator{
9595
"presto_cpp.num_tasks_with_stuck_operator"};
96-
constexpr folly::StringPiece kCounterNumCancelledTasksByStuckDriver{
96+
constexpr std::string_view kCounterNumCancelledTasksByStuckDriver{
9797
"presto_cpp.num_cancelled_tasks_by_stuck_driver"};
98-
constexpr folly::StringPiece kCounterNumTasksDeadlock{
98+
constexpr std::string_view kCounterNumTasksDeadlock{
9999
"presto_cpp.num_tasks_deadlock"};
100-
constexpr folly::StringPiece kCounterNumTaskManagerLockTimeOut{
100+
constexpr std::string_view kCounterNumTaskManagerLockTimeOut{
101101
"presto_cpp.num_tasks_manager_lock_timeout"};
102102

103-
constexpr folly::StringPiece kCounterNumQueuedDrivers{
103+
constexpr std::string_view kCounterNumQueuedDrivers{
104104
"presto_cpp.num_queued_drivers"};
105-
constexpr folly::StringPiece kCounterNumOnThreadDrivers{
105+
constexpr std::string_view kCounterNumOnThreadDrivers{
106106
"presto_cpp.num_on_thread_drivers"};
107-
constexpr folly::StringPiece kCounterNumSuspendedDrivers{
107+
constexpr std::string_view kCounterNumSuspendedDrivers{
108108
"presto_cpp.num_suspended_drivers"};
109-
constexpr folly::StringPiece kCounterNumBlockedWaitForConsumerDrivers{
109+
constexpr std::string_view kCounterNumBlockedWaitForConsumerDrivers{
110110
"presto_cpp.num_blocked_wait_for_consumer_drivers"};
111-
constexpr folly::StringPiece kCounterNumBlockedWaitForSplitDrivers{
111+
constexpr std::string_view kCounterNumBlockedWaitForSplitDrivers{
112112
"presto_cpp.num_blocked_wait_for_split_drivers"};
113-
constexpr folly::StringPiece kCounterNumBlockedWaitForProducerDrivers{
113+
constexpr std::string_view kCounterNumBlockedWaitForProducerDrivers{
114114
"presto_cpp.num_blocked_wait_for_producer_drivers"};
115-
constexpr folly::StringPiece kCounterNumBlockedWaitForJoinBuildDrivers{
115+
constexpr std::string_view kCounterNumBlockedWaitForJoinBuildDrivers{
116116
"presto_cpp.num_blocked_wait_for_join_build_drivers"};
117-
constexpr folly::StringPiece kCounterNumBlockedWaitForJoinProbeDrivers{
117+
constexpr std::string_view kCounterNumBlockedWaitForJoinProbeDrivers{
118118
"presto_cpp.num_blocked_wait_for_join_probe_drivers"};
119-
constexpr folly::StringPiece kCounterNumBlockedWaitForMergeJoinRightSideDrivers{
119+
constexpr std::string_view kCounterNumBlockedWaitForMergeJoinRightSideDrivers{
120120
"presto_cpp.num_blocked_wait_for_merge_join_right_side_drivers"};
121-
constexpr folly::StringPiece kCounterNumBlockedWaitForMemoryDrivers{
121+
constexpr std::string_view kCounterNumBlockedWaitForMemoryDrivers{
122122
"presto_cpp.num_blocked_wait_for_memory_drivers"};
123-
constexpr folly::StringPiece kCounterNumBlockedWaitForConnectorDrivers{
123+
constexpr std::string_view kCounterNumBlockedWaitForConnectorDrivers{
124124
"presto_cpp.num_blocked_wait_for_connector_drivers"};
125-
constexpr folly::StringPiece kCounterNumBlockedYieldDrivers{
125+
constexpr std::string_view kCounterNumBlockedYieldDrivers{
126126
"presto_cpp.num_blocked_yield_drivers"};
127-
constexpr folly::StringPiece kCounterNumStuckDrivers{
127+
constexpr std::string_view kCounterNumStuckDrivers{
128128
"presto_cpp.num_stuck_drivers"};
129129

130130
/// Export 100 if worker is overloaded in terms of memory, 0 otherwise.
131-
constexpr folly::StringPiece kCounterOverloadedMem{"presto_cpp.overloaded_mem"};
131+
constexpr std::string_view kCounterOverloadedMem{"presto_cpp.overloaded_mem"};
132132
/// Export 100 if worker is overloaded in terms of CPU, 0 otherwise.
133-
constexpr folly::StringPiece kCounterOverloadedCpu{"presto_cpp.overloaded_cpu"};
133+
constexpr std::string_view kCounterOverloadedCpu{"presto_cpp.overloaded_cpu"};
134134
/// Export 100 if worker is overloaded in terms of memory or CPU, 0 otherwise.
135-
constexpr folly::StringPiece kCounterOverloaded{"presto_cpp.overloaded"};
135+
constexpr std::string_view kCounterOverloaded{"presto_cpp.overloaded"};
136136
/// Worker exports the average time tasks spend in the queue (considered
137137
/// planned) in milliseconds.
138-
constexpr folly::StringPiece kCounterTaskPlannedTimeMs{
138+
constexpr std::string_view kCounterTaskPlannedTimeMs{
139139
"presto_cpp.task_planned_time_ms"};
140140
/// Exports the current overloaded duration in seconds or 0 if not currently
141141
/// overloaded.
@@ -144,36 +144,36 @@ constexpr folly::StringPiece kCounterOverloadedDurationSec{
144144

145145
/// Number of total OutputBuffer managed by all
146146
/// OutputBufferManager
147-
constexpr folly::StringPiece kCounterTotalPartitionedOutputBuffer{
147+
constexpr std::string_view kCounterTotalPartitionedOutputBuffer{
148148
"presto_cpp.num_partitioned_output_buffer"};
149149
/// Latency in millisecond of the get data call of a
150150
/// OutputBufferManager.
151-
constexpr folly::StringPiece kCounterPartitionedOutputBufferGetDataLatencyMs{
151+
constexpr std::string_view kCounterPartitionedOutputBufferGetDataLatencyMs{
152152
"presto_cpp.partitioned_output_buffer_get_data_latency_ms"};
153153

154154
/// ================== OS Counters =================
155155

156156
/// User CPU time of the presto_server process in microsecond since the process
157157
/// start.
158-
constexpr folly::StringPiece kCounterOsUserCpuTimeMicros{
158+
constexpr std::string_view kCounterOsUserCpuTimeMicros{
159159
"presto_cpp.os_user_cpu_time_micros"};
160160
/// System CPU time of the presto_server process in microsecond since the
161161
/// process start.
162-
constexpr folly::StringPiece kCounterOsSystemCpuTimeMicros{
162+
constexpr std::string_view kCounterOsSystemCpuTimeMicros{
163163
"presto_cpp.os_system_cpu_time_micros"};
164164
/// Total number of soft page faults of the presto_server process in microsecond
165165
/// since the process start.
166-
constexpr folly::StringPiece kCounterOsNumSoftPageFaults{
166+
constexpr std::string_view kCounterOsNumSoftPageFaults{
167167
"presto_cpp.os_num_soft_page_faults"};
168168
/// Total number of hard page faults of the presto_server process in microsecond
169169
/// since the process start.
170-
constexpr folly::StringPiece kCounterOsNumHardPageFaults{
170+
constexpr std::string_view kCounterOsNumHardPageFaults{
171171
"presto_cpp.os_num_hard_page_faults"};
172172
/// Total number of voluntary context switches in the presto_server process.
173-
constexpr folly::StringPiece kCounterOsNumVoluntaryContextSwitches{
173+
constexpr std::string_view kCounterOsNumVoluntaryContextSwitches{
174174
"presto_cpp.os_num_voluntary_context_switches"};
175175
/// Total number of involuntary context switches in the presto_server process.
176-
constexpr folly::StringPiece kCounterOsNumForcedContextSwitches{
176+
constexpr std::string_view kCounterOsNumForcedContextSwitches{
177177
"presto_cpp.os_num_forced_context_switches"};
178178

179179
/// ================== HiveConnector Counters ==================
@@ -210,30 +210,30 @@ constexpr std::string_view kCounterThreadPoolMaxIdleTimeNsFormat{
210210
"presto_cpp.{}.max_idle_time_ns"};
211211

212212
/// ================== EVB Counters ====================
213-
constexpr folly::StringPiece kCounterExchangeIoEvbViolation{
213+
constexpr std::string_view kCounterExchangeIoEvbViolation{
214214
"presto_cpp.exchange_io_evb_violation_count"};
215-
constexpr folly::StringPiece kCounterHttpServerIoEvbViolation{
215+
constexpr std::string_view kCounterHttpServerIoEvbViolation{
216216
"presto_cpp.http_server_io_evb_violation_count"};
217217

218218
/// ================== Memory Pushback Counters =================
219219

220220
/// Number of times memory pushback mechanism is triggered.
221-
constexpr folly::StringPiece kCounterMemoryPushbackCount{
221+
constexpr std::string_view kCounterMemoryPushbackCount{
222222
"presto_cpp.memory_pushback_count"};
223223
/// Latency distribution of each memory pushback run in range of [0, 100s] and
224224
/// reports P50, P90, P99, and P100.
225-
constexpr folly::StringPiece kCounterMemoryPushbackLatencyMs{
225+
constexpr std::string_view kCounterMemoryPushbackLatencyMs{
226226
"presto_cpp.memory_pushback_latency_ms"};
227227
/// Distribution of actual reduction in memory usage achieved by each memory
228228
/// pushback attempt. This is to gauge its effectiveness. In range of [0, 15GB]
229229
/// with 150 buckets and reports P50, P90, P99, and P100.
230-
constexpr folly::StringPiece kCounterMemoryPushbackReductionBytes{
230+
constexpr std::string_view kCounterMemoryPushbackReductionBytes{
231231
"presto_cpp.memory_pushback_reduction_bytes"};
232232
/// Distribution of expected reduction in memory usage achieved by each memory
233233
/// pushback attempt. This is to gauge its effectiveness. In range of [0, 15GB]
234234
/// with 150 buckets and reports P50, P90, P99, and P100. The expected reduction
235235
/// can be different as other threads might have allocated memory in the
236236
/// meantime.
237-
constexpr folly::StringPiece kCounterMemoryPushbackExpectedReductionBytes{
237+
constexpr std::string_view kCounterMemoryPushbackExpectedReductionBytes{
238238
"presto_cpp.memory_pushback_expected_reduction_bytes"};
239239
} // namespace facebook::presto

0 commit comments

Comments
 (0)