Add Cluster Tag#128
Conversation
simoneves
left a comment
There was a problem hiding this comment.
Seems fine, but what reads this? Is it related to your other Presto PR?
The PR that uses this flag and UI changes to display it linked in the PR description 😄 |
Ugh. Sorry. My brain was fuzz yesterday. |
|
@johallar I guess my remaining confusion is that the name |
Yeah, for this first version the idea was to keep it very generic and user defined. Second steps would default it in a smart way based on the cluster configuration. Still a little unclear myself how that defaulting will work |
Adds cluster tag config to our velox-testing start scripts for java, cpu, gpu.
Sister PR to prestodb/presto#26485, which will display this tag in the UI so we can easily identify which type of cluster we have created.