[hotfix] Remove presto node.properties files

apache / incubator-xtable

Apache XTable (incubating) is a cross-table converter for lakehouse table formats that facilitates interoperability across data processing systems and query engines.

https://xtable.apache.org/

Apache License 2.0

850 stars 139 forks source link

[hotfix] Remove presto node.properties files #467

Open daragu opened 3 months ago

daragu commented 3 months ago

Important Read

hotfix

What is the purpose of the pull request

The presto/node.properties will be changed after run the docker demo, and this file is useless, we can delete it.

On branch presto
Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
        modified:   presto/node.properties

These are generated by presto docker container.

Verify this pull request

local.

ghost commented 3 months ago

CI report:

0ba2e59a49332f36aa0caa9dece293e7eefb6d64 Azure: SUCCESS

Bot commands

@xtable-bot supports the following commands: - `@xtable-bot run azure` re-run the last Azure build

kristgpt commented 3 months ago

@daragu why so many node.id=presto records are produced.

daragu commented 3 months ago

@daragu why so many node.id=presto records are produced.

Hi @kristgpt, these records are generated after execute start_demo.sh every time.