-
Notifications
You must be signed in to change notification settings - Fork 77
[NSE-317]fix columnar cache #346
[NSE-317]fix columnar cache #346
Conversation
Thanks for opening a pull request! Could you open an issue for this pull request on Github Issues? https://github.com/oap-project/native-sql-engine/issues Then could you also rename commit message and pull request title in the following format?
See also: |
@rui-mo , please take a check |
verified on my env. This pr cleans up my test code and refresh the ut: xuechendi#3 |
49d4e03
to
acc178a
Compare
@zhouyuan , should be mergable
test: cache join result then do aggregate
|
* limitations under the License. | ||
*/ | ||
|
||
package org.apache.spark.sql.travis |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hi @xuechendi , we have changed this package "travis" into "nativesql", could you also put this file into nativesql folder?
This file should also be renamed into NativeCachedTableSuite.scala.
Signed-off-by: Chendi Xue <chendi.xue@intel.com>
…elease function Signed-off-by: Chendi Xue <chendi.xue@intel.com>
Signed-off-by: Chendi Xue <chendi.xue@intel.com>
Signed-off-by: Chendi Xue <chendi.xue@intel.com>
Signed-off-by: Chendi Xue <chendi.xue@intel.com>
Signed-off-by: Chendi Xue <chendi.xue@intel.com>
…zedCacheEntry to OffHeap Signed-off-by: Chendi Xue <chendi.xue@intel.com>
Signed-off-by: Chendi Xue <chendi.xue@intel.com>
Signed-off-by: Chendi Xue <chendi.xue@intel.com>
7c115dc
to
7fb4ec3
Compare
This PR is aim to fix current UT issues related with RDD cache
TODO:
need to back port spark PR's for manual close cached blocks, https://issues.apache.org/jira/browse/SPARK-35396
need to test with our jupyter test(now only UT tested)
Fixed: #317