Skip to content

Commit f7103fe

Browse files
joegswapsmagic
authored andcommitted
[native] Fix presto_cpp includes (prestodb#24625)
Summary: simple fix, remove unused include & move #include "github/presto-trunk/presto-native-execution/presto_cpp/main/common/Utils.h" to the two cpp files that directly use it Reviewed By: swapsmagic, karteekmurthys Differential Revision: D70195333
1 parent 40fe92e commit f7103fe

File tree

3 files changed

+2
-2
lines changed

3 files changed

+2
-2
lines changed

presto-native-execution/presto_cpp/main/types/PrestoToVeloxExpr.cpp

+1
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
*/
1414

1515
#include "presto_cpp/main/types/PrestoToVeloxExpr.h"
16+
#include "presto_cpp/main/common/Utils.h"
1617
#include <boost/algorithm/string/case_conv.hpp>
1718
#include "presto_cpp/main/common/Configs.h"
1819
#include "presto_cpp/presto_protocol/Base64Util.h"

presto-native-execution/presto_cpp/main/types/PrestoToVeloxExpr.h

-2
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,6 @@
1414
#pragma once
1515

1616
#include <stdexcept>
17-
#include "presto_cpp/main/common/Configs.h"
18-
#include "presto_cpp/main/common/Utils.h"
1917
#include "presto_cpp/main/types/TypeParser.h"
2018
#include "presto_cpp/presto_protocol/core/presto_protocol_core.h"
2119
#include "velox/core/Expressions.h"

presto-native-execution/presto_cpp/main/types/PrestoToVeloxQueryPlan.cpp

+1
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@
3333
#include "presto_cpp/main/operators/ShuffleRead.h"
3434
#include "presto_cpp/main/operators/ShuffleWrite.h"
3535
#include "presto_cpp/main/types/TypeParser.h"
36+
#include "presto_cpp/main/common/Utils.h"
3637

3738
using namespace facebook::velox;
3839
using namespace facebook::velox::exec;

0 commit comments

Comments
 (0)