Github user jacek-lewandowski commented on a diff in the pull request:
https://github.com/apache/spark/pull/9299#discussion_r43393292
--- Diff:
core/src/main/scala/org/apache/spark/deploy/master/ApplicationInfo.scala ---
@@ -32,7 +32,8 @@ private[spark] class ApplicationInfo(
val desc: ApplicationDescription,
val submitDate: Date,
val driver: RpcEndpointRef,
- defaultCores: Int)
+ defaultCores: Int,
+ var appUiUrl: String)
--- End diff --
Yes, I moved it to that class because it represent a mutable state of
application in the Master - and if the appUiUrl changes, it is due to
application state change. This class has many more mutable fields so it doesn't
make it worse. The main purpose of this ticket was to move out mutable fields
from `ApplicationDescription` which is used to transfer data from one process
to another and by definition it should be immutable.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
