feat: add sampling

[olzhik11]

Important

Add sampling to process_batch in consumer.rs using TRACE_SUMMARY_SAMPLE_RATE to control trace processing.

• Behavior:
  • Adds sampling to process_batch in consumer.rs using TRACE_SUMMARY_SAMPLE_RATE environment variable.
  • Uses rand::rngs::StdRng to generate random numbers for sampling decision.
  • Default sample rate is 0.25, clamped between 0.0 and 1.0.
• Error Handling:
  • Logs error if pushing trace completion to summary queue fails.
• Misc:
  • Imports rand::{Rng, SeedableRng} in consumer.rs.

^{This description was created by for 5a023be. You can customize this summary. It will automatically update as commits are pushed.}

[ellipsis-dev]

Caution

Changes requested ❌

Reviewed everything up to 5a023be in 2 minutes and 3 seconds. Click for details.

• Reviewed 51 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 2 draft comments. View those below.
• Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

1. app-server/src/traces/consumer.rs:345

• Draft comment:
  It appears that rand::rng() might be a typo. Typically, to obtain a random number generator one might use rand::thread_rng() instead. Please verify whether rand::rng() is the intended function.
• Reason this comment was not posted:
  Marked as duplicate.

2. app-server/src/traces/consumer.rs:349

• Draft comment:
  The method random_range does not seem standard. Usually, the method from the rand crate is gen_range. Consider replacing rng.random_range(0.0..1.0) with rng.gen_range(0.0..1.0) if appropriate.
• Reason this comment was not posted:
  Marked as duplicate.

Workflow ID: wflow_xRrrFWOK2jmQk1Fo

^{You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.}

[ellipsis-dev]

The RNG initialization is incorrect. Replace rand::rng() with a valid RNG, e.g., use rand::thread_rng() or StdRng::from_entropy().unwrap(), to properly seed StdRng.

Suggested change

	let mut rng = rand::rngs::StdRng::from_rng(&mut rand::rng());
	let mut rng = rand::rngs::StdRng::from_entropy();

[olzhik11]

hmm there is no from_entropy method on it

[ellipsis-dev]

Use the correct RNG method. Replace rng.random_range(0.0..1.0) with rng.gen_range(0.0..1.0) or consider using rng.gen_bool(sample_rate) for clarity.

Suggested change

	if rng.random_range(0.0..1.0) < sample_rate {
	if rng.gen_range(0.0..1.0) < sample_rate {

[olzhik11]

random_range is the right one