(Logs) Build transactions batch

[polydez]

Added logs for transaction batch building flow.

Sample output:

2024-01-29T09:01:42.062953Z  INFO block_producer:try_build_batches: miden-block-producer: block-producer/src/txqueue/mod.rs:162: new
2024-01-29T09:01:42.063159Z  INFO block_producer:try_build_batches:batch_builder: miden-block-producer: block-producer/src/txqueue/mod.rs:192: new
2024-01-29T09:01:42.063304Z  INFO block_producer:try_build_batches:batch_builder:build_batch: miden-block-producer: block-producer/src/batch_builder/mod.rs:116: new
2024-01-29T09:01:42.063354Z  INFO block_producer:try_build_batches:batch_builder:build_batch: miden-block-producer: block-producer/src/batch_builder/mod.rs:123: Building a transaction batch, num_txs: 1
2024-01-29T09:01:42.063412Z  INFO block_producer:try_build_batches:batch_builder:build_batch:new_batch: miden-block-producer: block-producer/src/batch_builder/batch.rs:41: new
2024-01-29T09:01:42.072082Z  INFO block_producer:try_build_batches:batch_builder:build_batch:new_batch: miden-block-producer: block-producer/src/batch_builder/batch.rs:41: close, time.busy: 8.14ms, time.idle: 526µs
2024-01-29T09:01:42.072189Z  INFO block_producer:try_build_batches:batch_builder:build_batch: miden-block-producer: block-producer/src/batch_builder/mod.rs:128: Transaction batch built
2024-01-29T09:01:42.072302Z  INFO block_producer:try_build_batches:batch_builder:build_batch: miden-block-producer: block-producer/src/batch_builder/mod.rs:116: close, time.busy: 8.95ms, time.idle: 50.4µs batch_id: "0x8a73a469f0af54c4c32588af08f7fd840f80a25e690a3247f252500fddc6f534"
2024-01-29T09:01:42.092294Z  INFO block_producer:try_build_batches:batch_builder: miden-block-producer: block-producer/src/txqueue/mod.rs:192: close, time.busy: 29.0ms, time.idle: 141µs
2024-01-29T09:01:42.092359Z  INFO block_producer:try_build_batches: miden-block-producer: block-producer/src/txqueue/mod.rs:162: close, time.busy: 174µs, time.idle: 29.3ms

[bobbinth]

Thank you! Looks good! I added some comments inline - most of them are related to removing some intermediary spans which I don't think add much value. Basically, in my mind the deepest span path could look like so:

block_producer:try_build_batches:build_batch:new_batch

As compared to the current:

block_producer::serve:transaction_queue:run:try_build_batches:block_producer::batch_builder::build_batch:build_batch:new

[bobbinth]

Thank you! Looks good! I left a couple of small comments inline. After these, we are good to merge.

[bobbinth]

Here, I would do something like:

but.extend_from_slice(&tx.id().as_bytes());