Skip to content

Commit

Permalink
Disable TBB by default
Browse files Browse the repository at this point in the history
This disables TBB by default in the bootstrap script.

Using our `/test/benchmarking` benchmarks, there is no discernible difference
between the end-to-end execution times or the individual execution times of the
parallel functions. For example, here is are the results of the `bench_large_io`
between the two:

// Without TBB
end-to-end: 267829 ms
`parallel_for`: 271120 ms
`parallel_for_2d`: 104 ms
`parallel_sort`: 14447 ms

// With TBB
end-to-end: 269911 ms
`parallel_for`: 273447 ms
`parallel_for_2d`: 107 ms
`parallel_sort`: 14382 ms

Note that the "end-to-end" was measured with a different timing mechanism than
the individual function timers. The individual function timers are also
cumulative per thread.
  • Loading branch information
Joe Maley committed Aug 10, 2020
1 parent 4878ae6 commit 23f3210
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 2 deletions.
1 change: 1 addition & 0 deletions HISTORY.md
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@
## Deprecations

* Support for MacOS older than 10.13 is being dropped when using the AWS SDK. Prebuilt Binaries now target 10.13 [#1753](https://github.com/TileDB-Inc/TileDB/pull/1753)
* Use of Intel's Thread Building Blocks (TBB) will be discontinued in the future. It is now disabled by default [#1762](https://github.com/TileDB-Inc/TileDB/pull/1762)

## Bug fixes

Expand Down
1 change: 0 additions & 1 deletion azure-pipelines.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,6 @@ stages:
linux_asan:
imageName: 'ubuntu-16.04'
TILEDB_CI_ASAN: ON
TILEDB_TBB: OFF
TILEDB_SERIALIZATION: ON
CXX: g++-7
linux_serialization:
Expand Down
2 changes: 1 addition & 1 deletion bootstrap
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ tiledb_werror="ON"
tiledb_tests="ON"
tiledb_cpp_api="ON"
tiledb_force_all_deps="OFF"
tiledb_tbb="ON"
tiledb_tbb="OFF"
tiledb_stats="ON"
tiledb_static="OFF"
tiledb_disable_avx2=""
Expand Down

0 comments on commit 23f3210

Please sign in to comment.