Skip to content

Commit ea85f4d

Browse files
committed
chore: idiomatic and optimized
1 parent e6c993d commit ea85f4d

File tree

4 files changed

+126
-143
lines changed

4 files changed

+126
-143
lines changed

src/handlers/http/logstream.rs

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -496,7 +496,7 @@ pub async fn delete_stream_hot_tier(
496496
}
497497

498498
pub async fn get_affected_resources(
499-
stream_name: Path<String>
499+
stream_name: Path<String>,
500500
) -> Result<impl Responder, StreamError> {
501501
let stream_name = stream_name.into_inner();
502502

@@ -508,10 +508,8 @@ pub async fn get_affected_resources(
508508
}
509509

510510
match LogstreamAffectedResources::load(&stream_name).await {
511-
Ok(affected_resources)
512-
=> Ok((web::Json(affected_resources), StatusCode::OK)),
513-
Err(err)
514-
=> Err(err.into())
511+
Ok(affected_resources) => Ok((web::Json(affected_resources), StatusCode::OK)),
512+
Err(err) => Err(err.into()),
515513
}
516514
}
517515

@@ -531,13 +529,13 @@ pub mod error {
531529
use http::StatusCode;
532530

533531
use crate::{
534-
hottier::HotTierError,
535-
metastore::MetastoreError,
536-
parseable::StreamNotFound,
537-
storage::ObjectStorageError,
532+
hottier::HotTierError,
533+
metastore::MetastoreError,
534+
parseable::StreamNotFound,
535+
storage::ObjectStorageError,
538536
validator::error::{
539537
AlertValidationError, HotTierValidationError, StreamNameValidationError,
540-
}
538+
},
541539
};
542540

543541
#[allow(unused)]

src/handlers/http/modal/ingest_server.rs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -273,12 +273,12 @@ impl IngestServer {
273273
),
274274
)
275275
.service(
276-
// GET "/logstream/{logstream}/affected-resources" ==> Get affected resources for given log stream
277-
web::resource("/affected-resources").route(
278-
web::get()
279-
.to(logstream::get_affected_resources)
280-
.authorize_for_resource(Action::GetLogstreamAffectedResources),
281-
),
276+
// GET "/logstream/{logstream}/affected-resources" ==> Get affected resources for given log stream
277+
web::resource("/affected-resources").route(
278+
web::get()
279+
.to(logstream::get_affected_resources)
280+
.authorize_for_resource(Action::GetLogstreamAffectedResources),
281+
),
282282
)
283283
.service(
284284
web::scope("/retention").service(

src/handlers/http/modal/query_server.rs

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -313,14 +313,6 @@ impl QueryServer {
313313
.authorize_for_resource(Action::GetStats),
314314
),
315315
)
316-
.service(
317-
// GET "/logstream/{logstream}/affected-resources" ==> Get affected resources for given log stream
318-
web::resource("/affected-resources").route(
319-
web::get()
320-
.to(logstream::get_affected_resources)
321-
.authorize_for_resource(Action::GetLogstreamAffectedResources),
322-
),
323-
)
324316
.service(
325317
web::resource("/retention")
326318
// PUT "/logstream/{logstream}/retention" ==> Set retention for given logstream

0 commit comments

Comments
 (0)