zhengruifeng opened a new pull request, #57674:
URL: https://github.com/apache/spark/pull/57674

   ### What changes were proposed in this pull request?
   
   This follow-up to #57665 applies its parameter-aware size estimation to 
`TrainValidationSplitModel`.
   
   It excludes the `estimator`, `estimatorParamMaps`, and `evaluator` params 
from metadata sizing, then explicitly accounts for the best model, validation 
metrics, and collected submodels. A regression test confirms that 
logger-initialized estimator state is not counted.
   
   ### Why are the changes needed?
   
   TrainValidationSplitModel retains the same kinds of tuning params as 
CrossValidatorModel. Their params and values may retain shared runtime state, 
such as SparkSession, which must not contribute to Spark Connect ML model-cache 
accounting.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   Static checks: `git diff --check`, non-ASCII scan, and changed-Scala 
line-length scan.
   
   The targeted Scala suite was not run locally.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   Generated-by: Codex GPT-5


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to