fix(rust): Functions for streaming require streaming feature

[eitsupi]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.84%. Comparing base (76b8f46) to head (25dcd96).
Report is 14 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #18602      +/-   ##
==========================================
- Coverage   79.84%   79.84%   -0.01%     
==========================================
  Files        1501     1501              
  Lines      201764   201836      +72     
  Branches     2869     2869              
==========================================
+ Hits       161107   161155      +48     
- Misses      40111    40135      +24     
  Partials      546      546              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[orlp]

I don't see why this fix would be necessary? The optimization flags for (new) streaming don't require those features.

[eitsupi]

I don't see why this fix would be necessary?

Unlike the following functions, these functions can be used without the feature flags, which can cause panic. (I encountered.)

polars/crates/polars-lazy/src/frame/mod.rs

Lines 166 to 178 in 9a256a3

	/// Toggle common subplan elimination optimization on or off
	#[cfg(feature = "cse")]
	pub fn with_comm_subplan_elim(mut self, toggle: bool) -> Self {
	self.opt_state.set(OptFlags::COMM_SUBPLAN_ELIM, toggle);
	self
	}
	/// Toggle common subexpression elimination optimization on or off
	#[cfg(feature = "cse")]
	pub fn with_comm_subexpr_elim(mut self, toggle: bool) -> Self {
	self.opt_state.set(OptFlags::COMM_SUBEXPR_ELIM, toggle);
	self
	}

I simply thought it would be more user-friendly to tie these to the flags for a consistency reason, so if you don't need it and don't mind causing a panic, then close it.

[orlp]

Alright if using them can cause a panic down the line I don't mind them being feature gated.