search

prestodb / presto

The official home of the Presto distributed SQL query engine for big data
http://prestodb.io
Apache License 2.0
15.97k stars 5.35k forks source link

[native] Update presto protocol #23643

Closed imjalpreet closed 2 weeks ago

imjalpreet commented 2 weeks ago

Description

Due to recent refactoring as part of the commit 9d463405

Motivation and Context

Below classes got moved from sql/planner to spi/plan

Fix failures when building presto protocol

./java-to-struct-json.py special/*.java special/*.inc -j | jq . > presto_protocol.json
Traceback (most recent call last):
  File "/Users/imjalpreet/Documents/OSS/presto/presto-native-execution/presto_cpp/presto_protocol/./java-to-struct-json.py", line 377, in <module>
    sys.exit(main())
  File "/Users/imjalpreet/Documents/OSS/presto/presto-native-execution/presto_cpp/presto_protocol/./java-to-struct-json.py", line 363, in main
    process_file(file, config, lang, subclasses, classes, depends)
  File "/Users/imjalpreet/Documents/OSS/presto/presto-native-execution/presto_cpp/presto_protocol/./java-to-struct-json.py", line 192, in process_file
    for line in preprocess_file(filepath):
  File "/Users/imjalpreet/Documents/OSS/presto/presto-native-execution/presto_cpp/presto_protocol/./java-to-struct-json.py", line 84, in preprocess_file
    text = util.file_read(filename)
  File "/Users/imjalpreet/Documents/OSS/presto/presto-native-execution/presto_cpp/presto_protocol/util.py", line 93, in file_read
    with open(filename) as file:
FileNotFoundError: [Errno 2] No such file or directory: '/Users/imjalpreet/Documents/OSS/presto/presto-main/src/main/java/com/facebook/presto/sql/planner/Partitioning.java'
== NO RELEASE NOTE ==
imjalpreet commented 2 weeks ago

@yingsu00 I have re-triggered the test, most likely it was a flaky test since I haven't made any Java changes in this PR.