pitrou commented on code in PR #13516:
URL: https://github.com/apache/arrow/pull/13516#discussion_r972775560
##########
cpp/src/arrow/memory_pool.cc:
##########
@@ -665,6 +665,20 @@ MemoryPool* default_memory_pool() {
Status jemalloc_set_decay_ms(int ms) {
return Status::Invalid("jemalloc support is not built");
}
+
+Status jemalloc_get_stat(const char* name, size_t* out) {
+ return Status::Invalid("jemalloc support is not built");
Review Comment:
TBH, I'd rather switch them all to `NotImplemented`, and reserve `Invalid`
for when the parameter given to the function is invalid.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
