Skip to content

Simplify run-trino script#12487

Merged
findepi merged 1 commit intotrinodb:masterfrom
nineinchnick:simplify-run-trino-script
May 20, 2022
Merged

Simplify run-trino script#12487
findepi merged 1 commit intotrinodb:masterfrom
nineinchnick:simplify-run-trino-script

Conversation

@nineinchnick
Copy link
Member

Description

Make run-script a bit shorter. This is visible to users in the container logs since tracing is enabled (set -x).

Is this change a fix, improvement, new feature, refactoring, or other?

Is this a change to the core query engine, a connector, client library, or the SPI interfaces? (be specific)

How would you describe this change to a non-technical end user or system administrator?

Related issues, pull requests, and links

Documentation

(x) No documentation is needed.
( ) Sufficient documentation is included in this PR.
( ) Documentation PR is available with #prnumber.
( ) Documentation issue #issuenumber is filed, and can be handled later.

Release notes

(x) No release notes entries required.
( ) Release notes entries required with the following suggested text:

# Section
* Fix some things. ({issue}`issuenumber`)

@cla-bot cla-bot bot added the cla-signed label May 20, 2022
@findepi findepi added the no-release-notes This pull request does not require release notes entry label May 20, 2022
@nineinchnick nineinchnick requested a review from findepi May 20, 2022 09:56
@nineinchnick nineinchnick force-pushed the simplify-run-trino-script branch from b75ac71 to 830273f Compare May 20, 2022 10:04
@nineinchnick nineinchnick force-pushed the simplify-run-trino-script branch from 830273f to ac71185 Compare May 20, 2022 12:27
@findepi findepi merged commit 2c45353 into trinodb:master May 20, 2022
@github-actions github-actions bot added this to the 382 milestone May 20, 2022
@nineinchnick nineinchnick deleted the simplify-run-trino-script branch May 22, 2022 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed no-release-notes This pull request does not require release notes entry

Development

Successfully merging this pull request may close these issues.

2 participants