incubator-xtable icon indicating copy to clipboard operation
incubator-xtable copied to clipboard

[hotfix] Remove presto node.properties files

Open daragu opened this issue 1 year ago • 3 comments

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.

daragu avatar Jun 13 '24 02:06 daragu

CI report:

  • 0ba2e59a49332f36aa0caa9dece293e7eefb6d64 Azure: SUCCESS
Bot commands @xtable-bot supports the following commands:
  • @xtable-bot run azure re-run the last Azure build

ghost avatar Jun 13 '24 02:06 ghost

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

kristgpt avatar Jun 17 '24 01:06 kristgpt

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

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

daragu avatar Jun 17 '24 06:06 daragu