Skip to content

Commit

Permalink
Cleanup span
Browse files Browse the repository at this point in the history
  • Loading branch information
lewiszlw committed Jan 30, 2024
1 parent 83d329c commit a7a6f4a
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 4 deletions.
2 changes: 0 additions & 2 deletions bustubx/src/database.rs
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,6 @@ impl Database {
}

pub fn run(&mut self, sql: &str) -> BustubxResult<Vec<Tuple>> {
let _db_run_span = span!(tracing::Level::INFO, "database.run", sql).entered();

let logical_plan = self.build_logical_plan(sql)?;
println!("{:?}", logical_plan);

Expand Down
1 change: 0 additions & 1 deletion bustubx/src/execution/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ pub struct ExecutionEngine<'a> {
}
impl ExecutionEngine<'_> {
pub fn execute(&mut self, plan: Arc<PhysicalPlan>) -> Vec<Tuple> {
let _execute_span = span!(tracing::Level::INFO, "executionengine.execute").entered();
plan.init(&mut self.context);
let mut result = Vec::new();
loop {
Expand Down
1 change: 0 additions & 1 deletion bustubx/src/parser/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ use sqlparser::{ast::Statement, dialect::PostgreSqlDialect, parser::Parser};
use tracing::span;

Check warning on line 3 in bustubx/src/parser/mod.rs

View workflow job for this annotation

GitHub Actions / Test Suite

unused import: `tracing::span`

pub fn parse_sql(sql: &str) -> BustubxResult<Vec<Statement>> {
let _parse_sql_span = span!(tracing::Level::INFO, "parse_sql", sql).entered();
let stmts = Parser::parse_sql(&PostgreSqlDialect {}, sql)?;
Ok(stmts)
}
Expand Down

0 comments on commit a7a6f4a

Please sign in to comment.