spark-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From JoshRosen <...@git.apache.org>
Subject [GitHub] spark pull request: [SPARK-8357] [SQL] Memory leakage on unsafe ag...
Date Tue, 23 Jun 2015 02:52:01 GMT
Github user JoshRosen commented on the pull request:

    https://github.com/apache/spark/pull/6810#issuecomment-114335130
  
    Also, if we do introduce a more general way of handling this sort of state cleanup as
part of the test runner, then I suspect we'll want to both remove a lot of the old try-finally
code and will have to fix any bugs due to tests that implicitly relied on the old behavior.
 Since that has the potential to grow to cover a lot more changes than this bugfix, I'd like
to tackle it separately.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


Mime
View raw message