aglinxinyuan opened a new issue, #4915: URL: https://github.com/apache/texera/issues/4915
`AddressInfo` (amber/src/main/scala/org/apache/texera/amber/engine/architecture/deploysemantics/AddressInfo.scala) is the case class the cluster scheduler uses to describe worker / controller node addresses. It has no test coverage today. Add an `AddressInfoSpec` that pins: - Both fields round-trip from the case-class constructor. - The order of `allAddresses` is preserved (the scheduler picks workers based on it). - Empty `allAddresses` is accepted (controller-only configurations). - The controller is allowed to also appear in `allAddresses` (collocated mode). - `copy()` lets a single field change while preserving the other. - Case-class equality with an `Array` field is reference-based, not element-wise — pin this so a future switch to (say) `Seq` doesn't silently flip the equality semantics for callers. -- 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]
