diff --git a/pageserver/src/http/routes.rs b/pageserver/src/http/routes.rs index 01f1514f0c..302fa524e7 100644 --- a/pageserver/src/http/routes.rs +++ b/pageserver/src/http/routes.rs @@ -96,10 +96,9 @@ fn get_include_non_incremental_logical_size(request: &Request) -> bool { .map(|v| { url::form_urlencoded::parse(v.as_bytes()) .into_owned() - .position(|(param, _)| param == "include-non-incremental-logical-size") + .any(|(param, _)| param == "include-non-incremental-logical-size") }) - .flatten() - .is_some() + .unwrap_or(false) } async fn branch_list_handler(request: Request) -> Result, ApiError> { diff --git a/pageserver/src/page_service.rs b/pageserver/src/page_service.rs index d7c0b1b2ee..f8053fadd1 100644 --- a/pageserver/src/page_service.rs +++ b/pageserver/src/page_service.rs @@ -630,7 +630,9 @@ impl postgres_backend::Handler for PageServerHandler { let tenantid = ZTenantId::from_str(caps.get(1).unwrap().as_str())?; - let branches = crate::branches::get_branches(self.conf, &tenantid, true)?; + // since these handlers for tenant/branch commands are deprecated (in favor of http based ones) + // just use false in place of include non incremental logical size + let branches = crate::branches::get_branches(self.conf, &tenantid, false)?; let branches_buf = serde_json::to_vec(&branches)?; pgb.write_message_noflush(&SINGLE_COL_ROWDESC)?