Skip to content

Commit 8449425

Browse files
committed
Address review feedback
1 parent 9961a3f commit 8449425

File tree

9 files changed

+475
-452
lines changed

9 files changed

+475
-452
lines changed

presto-native-execution/presto_cpp/main/CMakeLists.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ add_library(
4444
add_dependencies(
4545
presto_server_lib
4646
presto_operators
47-
presto_plan
47+
presto_velox_plan
4848
presto_protocol
4949
presto_types
5050
presto_thrift-cpp2
@@ -58,7 +58,7 @@ target_link_libraries(
5858
$<TARGET_OBJECTS:presto_type_converter>
5959
$<TARGET_OBJECTS:presto_types>
6060
$<TARGET_OBJECTS:presto_protocol>
61-
presto_plan
61+
presto_velox_plan
6262
presto_common
6363
presto_exception
6464
presto_expression_optimizer

presto-native-execution/presto_cpp/main/operators/tests/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ add_test(presto_operators_test presto_operators_test)
2727
target_link_libraries(
2828
presto_operators_test
2929
presto_connectors
30-
presto_plan
30+
presto_velox_plan
3131
presto_operators_plan_builder
3232
presto_operators
3333
presto_protocol

presto-native-execution/presto_cpp/main/plan/CMakeLists.txt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,9 @@
1010
# See the License for the specific language governing permissions and
1111
# limitations under the License.
1212

13-
add_library(presto_plan PrestoToVeloxQueryPlan.cpp VeloxPlanChecker.cpp)
13+
add_library(presto_velox_plan PrestoToVeloxQueryPlan.cpp VeloxPlanChecker.cpp)
1414
target_link_libraries(
15-
presto_plan
15+
presto_velox_plan
1616
presto_velox_expr_conversion
1717
presto_connectors
1818
presto_operators
@@ -24,4 +24,4 @@ target_link_libraries(
2424
velox_core
2525
)
2626

27-
set_property(TARGET presto_plan PROPERTY JOB_POOL_LINK presto_link_job_pool)
27+
set_property(TARGET presto_velox_plan PROPERTY JOB_POOL_LINK presto_link_job_pool)

0 commit comments

Comments
 (0)