diff options
| author | Xiangrui Meng <meng@databricks.com> | 2016-12-09 07:51:46 -0800 |
|---|---|---|
| committer | Xiangrui Meng <meng@databricks.com> | 2016-12-09 07:51:46 -0800 |
| commit | fd48d80a6145ea94f03e7fc6e4d724a0fbccac58 (patch) | |
| tree | 000474455efdb56d0bf6417c3e365c78b202a5a7 /assembly | |
| parent | b162cc0c2810c1a9fa2eee8e664ffae84f9eea11 (diff) | |
| download | spark-fd48d80a6145ea94f03e7fc6e4d724a0fbccac58.tar.gz spark-fd48d80a6145ea94f03e7fc6e4d724a0fbccac58.tar.bz2 spark-fd48d80a6145ea94f03e7fc6e4d724a0fbccac58.zip | |
[SPARK-17822][R] Make JVMObjectTracker a member variable of RBackend
## What changes were proposed in this pull request?
* This PR changes `JVMObjectTracker` from `object` to `class` and let its instance associated with each RBackend. So we can manage the lifecycle of JVM objects when there are multiple `RBackend` sessions. `RBackend.close` will clear the object tracker explicitly.
* I assume that `SQLUtils` and `RRunner` do not need to track JVM instances, which could be wrong.
* Small refactor of `SerDe.sqlSerDe` to increase readability.
## How was this patch tested?
* Added unit tests for `JVMObjectTracker`.
* Wait for Jenkins to run full tests.
Author: Xiangrui Meng <meng@databricks.com>
Closes #16154 from mengxr/SPARK-17822.
Diffstat (limited to 'assembly')
0 files changed, 0 insertions, 0 deletions