Clone Tools
  • last updated a few minutes ago
Constraints: committers
Constraints: files
Constraints: dates
DRILL-5783, DRILL-5841, DRILL-5894: Rationalize test temp directories

This change includes:


- A unit test is created for the priority queue in the TopN operator.

- The code generation classes passed around a completely unused function registry reference in some places so it is removed.

- The priority queue had unused parameters for some of its methods so it is removed.


- Created standardized temp directory classes DirTestWatcher, SubDirTestWatcher, and BaseDirTestWatcher. And updated all unit tests to use them.


- Removed the dfs_test storage plugin for tests and replaced it with the already existing dfs storage plugin.


- General code cleanup.

- Removed unnecessary use of String.format in the tests.

This closes #984

  1. … 365 more files in changeset.
DRILL-3147: tpcds-sf1-parquet query 73 causes memory leak

- each time a fragment A sends a "receiver finished" to fragment B, fragment B id will be

added to FragmentContext.ignoredSenders list

- refactored UnorderedReceiverBatch.informSenders() and MergingRecordBatch.informSenders()

by moving this method to FragmentContext

- DataServer.send() uses FragmentContext.ignoredSenders to decide if a batch should be

passed to the fragment or discarded right away

- BaseRawBatchBuffer methods enqueue() and kill() are now synchronized

- TestTpcdsSf1Leak test reproduces the leak, it's ignored by default because it requires

a large dataset

  1. … 5 more files in changeset.