GitHub user ash211 opened a pull request:
https://github.com/apache/spark/pull/19153
SPARK-21941 Stop storing unused attemptId in SQLTaskMetrics
## What changes were proposed in this pull request?
In a driver heap dump containing 390,105 instances of SQLTaskMetrics this
would have saved approximately 3.2MB of memory.
Since we're not getting any benefit from storing this unused value, let's
eliminate it until a future PR makes use of it.
## How was this patch tested?
Existing unit tests
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ash211/spark aash/trim-sql-listener
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/19153.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #19153
----
commit 4490740b93d3b5d8f1f46e49a608115fe0d37a57
Author: Andrew Ash <and...@andrewash.com>
Date: 2017-09-07T06:05:30Z
SPARK-21941 Stop storing unused attemptId in SQLTaskMetrics
In a driver heap dump containing 390,105 instances of SQLTaskMetrics this
would have saved approximately 3.2MB of memory.
Since we're not getting any benefit from storing this unused value, let's
eliminate it until a future PR makes use of it.
----
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
