Skip to content

Commit

Permalink
Disable TBB by default (#1762)
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.

Co-authored-by: Joe Maley <[email protected]>
  • Loading branch information
joe maley and Joe Maley authored Aug 11, 2020
1 parent 4728b34 commit 466d5ef
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ option(TILEDB_HDFS "Enables HDFS support using the official Hadoop JNI bindings"
option(TILEDB_WERROR "Enables the -Werror flag during compilation." ON)
option(TILEDB_CPP_API "Enables building of the TileDB C++ API." ON)
option(TILEDB_CMAKE_IDE "(Used for CLion builds). Disables superbuild and sets the EP install dir." OFF)
option(TILEDB_TBB "Enables use of TBB for parallelization." ON)
option(TILEDB_TBB "Enables use of TBB for parallelization." OFF)
option(TILEDB_TBB_SHARED "If TBB is enabled, builds TBB as a shared library instead of static." OFF)
option(TILEDB_STATS "Enables internal TileDB statistics gathering." ON)
option(TILEDB_STATIC "Enables building TileDB as a static library." OFF)
Expand Down
1 change: 1 addition & 0 deletions HISTORY.md
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,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 466d5ef

Please sign in to comment.