Skip to content

Improve error messages#11446

Merged
akuzm merged 1 commit intomasterfrom
aku/better-error-messages
Jun 5, 2020
Merged

Improve error messages#11446
akuzm merged 1 commit intomasterfrom
aku/better-error-messages

Conversation

@akuzm
Copy link
Copy Markdown
Contributor

@akuzm akuzm commented Jun 4, 2020

Also support fmt for exception messages, the same way we use it in
logging macros.

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Also support fmt for exception messages, the same way we use it in
logging macros.
@blinkov blinkov added the pr-not-for-changelog This PR should not be mentioned in the changelog label Jun 4, 2020
@alexey-milovidov alexey-milovidov self-assigned this Jun 4, 2020
@akuzm akuzm merged commit 7db8edf into master Jun 5, 2020
@akuzm akuzm deleted the aku/better-error-messages branch June 5, 2020 10:11
akuzm added a commit that referenced this pull request Jul 10, 2020
Improve error messages

(cherry picked from commit 7db8edf)
akuzm added a commit that referenced this pull request Jul 14, 2020
* [wip] add fmtlib to 20.3

* Merge pull request #11446 from ClickHouse/aku/better-error-messages

Improve error messages

(cherry picked from commit 7db8edf)

* fixup

* Merge pull request #11076 from ClickHouse/aku/join-error-messages

Better error messages about joins

(cherry picked from commit 4b1bb43)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-docs-needed pr-not-for-changelog This PR should not be mentioned in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants