pageserver: rename data_api to page_api

This commit is contained in:
Erik Grinaker
2025-04-29 15:54:37 +02:00
parent 93eb7bb6b8
commit 0f520d79ab
18 changed files with 36 additions and 61 deletions

10
Cargo.lock generated
View File

@@ -1353,7 +1353,7 @@ dependencies = [
"neonart",
"nix 0.27.1",
"pageserver_client_grpc",
"pageserver_data_api",
"pageserver_page_api",
"prost 0.13.3",
"thiserror 1.0.69",
"tokio",
@@ -4334,7 +4334,7 @@ dependencies = [
"pageserver_api",
"pageserver_client",
"pageserver_client_grpc",
"pageserver_data_api",
"pageserver_page_api",
"rand 0.8.5",
"reqwest",
"serde",
@@ -4412,7 +4412,7 @@ dependencies = [
"pageserver_api",
"pageserver_client",
"pageserver_compaction",
"pageserver_data_api",
"pageserver_page_api",
"peekable",
"pem",
"pin-project-lite",
@@ -4529,7 +4529,7 @@ dependencies = [
"bytes",
"futures",
"http 1.1.0",
"pageserver_data_api",
"pageserver_page_api",
"thiserror 1.0.69",
"tonic",
"tracing",
@@ -4559,7 +4559,7 @@ dependencies = [
]
[[package]]
name = "pageserver_data_api"
name = "pageserver_page_api"
version = "0.1.0"
dependencies = [
"prost 0.13.3",

View File

@@ -256,7 +256,7 @@ pageserver = { path = "./pageserver" }
pageserver_api = { version = "0.1", path = "./libs/pageserver_api/" }
pageserver_client = { path = "./pageserver/client" }
pageserver_client_grpc = { path = "./pageserver/client_grpc" }
pageserver_data_api = { path = "./pageserver/data_api" }
pageserver_page_api = { path = "./pageserver/page_api" }
pageserver_compaction = { version = "0.1", path = "./pageserver/compaction/" }
postgres_backend = { version = "0.1", path = "./libs/postgres_backend/" }
postgres_connection = { version = "0.1", path = "./libs/postgres_connection/" }

View File

@@ -79,7 +79,7 @@ url.workspace = true
walkdir.workspace = true
metrics.workspace = true
pageserver_api.workspace = true
pageserver_data_api.workspace = true
pageserver_page_api.workspace = true
pageserver_client.workspace = true # for ResponseErrorMessageExt TOOD refactor that
pageserver_compaction.workspace = true
pem.workspace = true

View File

@@ -11,4 +11,4 @@ thiserror.workspace = true
tonic.workspace = true
tracing.workspace = true
pageserver_data_api.workspace = true
pageserver_page_api.workspace = true

View File

@@ -12,14 +12,14 @@ use thiserror::Error;
use tonic::metadata::AsciiMetadataValue;
use tonic::transport::Channel;
use pageserver_data_api::model::*;
use pageserver_data_api::proto;
use pageserver_page_api::model::*;
use pageserver_page_api::proto;
type Shardno = u16;
use pageserver_data_api::client::PageServiceClient;
use pageserver_page_api::client::PageServiceClient;
type MyPageServiceClient = pageserver_data_api::client::PageServiceClient<
type MyPageServiceClient = pageserver_page_api::client::PageServiceClient<
tonic::service::interceptor::InterceptedService<tonic::transport::Channel, AuthInterceptor>,
>;

View File

@@ -1,5 +1,5 @@
[package]
name = "pageserver_data_api"
name = "pageserver_page_api"
version = "0.1.0"
edition = "2024"

View File

@@ -25,7 +25,7 @@ tokio-util.workspace = true
pageserver_client.workspace = true
pageserver_client_grpc.workspace = true
pageserver_data_api.workspace = true
pageserver_page_api.workspace = true
pageserver_api.workspace = true
utils = { path = "../../libs/utils/" }
workspace_hack = { version = "0.1", path = "../../workspace_hack" }

View File

@@ -9,7 +9,7 @@ use anyhow::Context;
use pageserver_api::shard::TenantShardId;
use pageserver_client::mgmt_api::ForceAwaitLogicalSize;
use pageserver_client::page_service::BasebackupRequest;
use pageserver_data_api::model::{GetBaseBackupRequest, RequestCommon};
use pageserver_page_api::model::{GetBaseBackupRequest, RequestCommon};
use rand::prelude::*;
use tokio::sync::Barrier;

View File

@@ -302,38 +302,11 @@ async fn main_impl(
Box::pin(async move {
if args.grpc_stream {
client_grpc_stream(
args,
worker_id,
ss,
cancel,
rps_period,
ranges,
weights,
)
.await
client_grpc_stream(args, worker_id, ss, cancel, rps_period, ranges, weights).await
} else if args.grpc {
client_grpc(
args,
worker_id,
ss,
cancel,
rps_period,
ranges,
weights,
)
.await
client_grpc(args, worker_id, ss, cancel, rps_period, ranges, weights).await
} else {
client_libpq(
args,
worker_id,
ss,
cancel,
rps_period,
ranges,
weights,
)
.await
client_libpq(args, worker_id, ss, cancel, rps_period, ranges, weights).await
}
})
};
@@ -522,8 +495,8 @@ async fn client_grpc(
let (rel_tag, block_no) = key
.to_rel_block()
.expect("we filter non-rel-block keys out above");
pageserver_data_api::model::GetPageRequest {
common: pageserver_data_api::model::RequestCommon {
pageserver_page_api::model::GetPageRequest {
common: pageserver_page_api::model::RequestCommon {
request_lsn: if rng.gen_bool(args.req_latest_probability) {
Lsn::MAX
} else {
@@ -531,7 +504,7 @@ async fn client_grpc(
},
not_modified_since_lsn: r.timeline_lsn,
},
rel: pageserver_data_api::model::RelTag {
rel: pageserver_page_api::model::RelTag {
spc_oid: rel_tag.spcnode,
db_oid: rel_tag.dbnode,
rel_number: rel_tag.relnode,
@@ -605,7 +578,9 @@ async fn client_grpc_stream(
usize::try_from(client_start.elapsed().as_micros() / period.as_micros()).unwrap();
if periods_passed_until_now > ticks_processed {
shared_state.live_stats.missed((periods_passed_until_now - ticks_processed) as u64);
shared_state
.live_stats
.missed((periods_passed_until_now - ticks_processed) as u64);
}
ticks_processed = periods_passed_until_now;
}
@@ -622,8 +597,8 @@ async fn client_grpc_stream(
let (rel_tag, block_no) = key
.to_rel_block()
.expect("we filter non-rel-block keys out above");
pageserver_data_api::model::GetPageRequest {
common: pageserver_data_api::model::RequestCommon {
pageserver_page_api::model::GetPageRequest {
common: pageserver_page_api::model::RequestCommon {
request_lsn: if rng.gen_bool(args.req_latest_probability) {
Lsn::MAX
} else {
@@ -631,7 +606,7 @@ async fn client_grpc_stream(
},
not_modified_since_lsn: r.timeline_lsn,
},
rel: pageserver_data_api::model::RelTag {
rel: pageserver_page_api::model::RelTag {
spc_oid: rel_tag.spcnode,
db_oid: rel_tag.dbnode,
rel_number: rel_tag.relnode,

View File

@@ -40,9 +40,9 @@ use tokio_util::sync::CancellationToken;
use futures::stream::StreamExt;
use pageserver_data_api::model;
use pageserver_data_api::proto::page_service_server::PageService;
use pageserver_data_api::proto::page_service_server::PageServiceServer;
use pageserver_page_api::model;
use pageserver_page_api::proto::page_service_server::PageService;
use pageserver_page_api::proto::page_service_server::PageServiceServer;
use anyhow::Context;
use bytes::BytesMut;
@@ -71,7 +71,7 @@ use postgres_ffi::pg_constants::DEFAULTTABLESPACE_OID;
use postgres_backend::AuthType;
pub use pageserver_data_api::proto;
pub use pageserver_page_api::proto;
pub(super) fn launch_compute_service_grpc_server(
tcp_connections_rx: tokio::sync::mpsc::Receiver<tokio::io::Result<tokio::net::TcpStream>>,

View File

@@ -26,7 +26,7 @@ tokio-epoll-uring.workspace = true
uring-common.workspace = true
pageserver_client_grpc.workspace = true
pageserver_data_api.workspace = true
pageserver_page_api.workspace = true
neonart.workspace = true
utils.workspace = true

View File

@@ -14,7 +14,7 @@
use utils::lsn::Lsn;
use crate::file_cache::{CacheBlock, FileCache};
use pageserver_data_api::model::RelTag;
use pageserver_page_api::model::RelTag;
use neonart;
use neonart::TreeInitStruct;

View File

@@ -4,7 +4,7 @@ type COid = u32;
// This conveniently matches PG_IOV_MAX
pub const MAX_GETPAGEV_PAGES: usize = 32;
use pageserver_data_api::model;
use pageserver_page_api::model;
#[repr(C)]
#[derive(Copy, Clone, Debug)]
@@ -12,7 +12,7 @@ pub enum NeonIORequest {
Empty,
// Read requests. These are C-friendly variants of the corresponding structs in
// pageserver_data_api::model.
// pageserver_page_api::model.
RelExists(CRelExistsRequest),
RelSize(CRelSizeRequest),
GetPageV(CGetPageVRequest),

View File

@@ -8,7 +8,7 @@ use crate::integrated_cache::{CacheResult, IntegratedCacheWriteAccess};
use crate::neon_request::{CGetPageVRequest, CPrefetchVRequest};
use crate::neon_request::{NeonIORequest, NeonIOResult};
use pageserver_client_grpc::PageserverClient;
use pageserver_data_api::model;
use pageserver_page_api::model;
use tokio::io::AsyncReadExt;
use tokio_epoll_uring::IoBuf;