Skip to content

Commit

Permalink
Fix format
Browse files Browse the repository at this point in the history
  • Loading branch information
luhenry committed Mar 6, 2024
1 parent f4772dd commit 72d5972
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 5 deletions.
5 changes: 3 additions & 2 deletions CMake/resolve_dependency_modules/folly/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,9 @@ if(VELOX_ENABLE_GPU)
set(cudacc_patch && git apply ${CMAKE_CURRENT_LIST_DIR}/folly-cudacc.patch)
endif()

set(VELOX_FOLLY_PATCH_COMMAND git apply ${CMAKE_CURRENT_LIST_DIR}/folly-no-export.patch
${glog_patch} ${cudacc_patch})
set(VELOX_FOLLY_PATCH_COMMAND
git apply ${CMAKE_CURRENT_LIST_DIR}/folly-no-export.patch ${glog_patch}
${cudacc_patch})

FetchContent_Declare(
folly
Expand Down
2 changes: 1 addition & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -373,7 +373,7 @@ if(${VELOX_ENABLE_GPU})
enable_language(CUDA)
# Determine CUDA_ARCHITECTURES automatically.
cmake_policy(SET CMP0104 NEW)
if (NOT DEFINED CMAKE_CUDA_ARCHITECTURES)
if(NOT DEFINED CMAKE_CUDA_ARCHITECTURES)
# it will fail later in the build otherwise
message(FATAL_ERROR "-DCMAKE_CUDA_ARCHITECTURES= must be set")
endif()
Expand Down
7 changes: 5 additions & 2 deletions velox/core/PlanNode.h
Original file line number Diff line number Diff line change
Expand Up @@ -2406,8 +2406,11 @@ struct fmt::formatter<facebook::velox::core::JoinType> : formatter<int> {
};

template <>
struct fmt::formatter<facebook::velox::core::AggregationNode::Step> : formatter<std::string> {
auto format(facebook::velox::core::AggregationNode::Step s, format_context& ctx) {
struct fmt::formatter<facebook::velox::core::AggregationNode::Step>
: formatter<std::string> {
auto format(
facebook::velox::core::AggregationNode::Step s,
format_context& ctx) {
return formatter<std::string>::format(
facebook::velox::core::mapAggregationStepToName(s), ctx);
}
Expand Down

0 comments on commit 72d5972

Please sign in to comment.