This is an automated email from the ASF dual-hosted git repository.
gurwls223 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new d5fa5cba4f5 [SPARK-39468][CORE][FOLLOWUP] Use `lazy val` for `host`
d5fa5cba4f5 is described below
commit d5fa5cba4f55df40a54cdffdf446b69e7aff3182
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Thu Jun 16 09:25:41 2022 +0900
[SPARK-39468][CORE][FOLLOWUP] Use `lazy val` for `host`
### What changes were proposed in this pull request?
This PR aims to use `lazy val host` instead of `val host`.
### Why are the changes needed?
To address the review comments about `RpcAddress` object size.
- https://github.com/apache/spark/pull/36868#discussion_r898235985
- https://github.com/apache/spark/pull/36868#discussion_r898332785
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Pass the CIs.
Closes #36882 from dongjoon-hyun/SPARK-39468-2.
Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Hyukjin Kwon <[email protected]>
---
core/src/main/scala/org/apache/spark/rpc/RpcAddress.scala | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/core/src/main/scala/org/apache/spark/rpc/RpcAddress.scala
b/core/src/main/scala/org/apache/spark/rpc/RpcAddress.scala
index 2a2d2051799..9b0739c9447 100644
--- a/core/src/main/scala/org/apache/spark/rpc/RpcAddress.scala
+++ b/core/src/main/scala/org/apache/spark/rpc/RpcAddress.scala
@@ -25,7 +25,7 @@ import org.apache.spark.util.Utils
*/
private[spark] case class RpcAddress(_host: String, port: Int) {
- val host: String = Utils.addBracketsIfNeeded(_host)
+ lazy val host: String = Utils.addBracketsIfNeeded(_host)
def hostPort: String = host + ":" + port
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
