paul-rogers opened a new pull request, #13778:
URL: https://github.com/apache/druid/pull/13778
This PR is a collection if minor edits done as part of the Catalog project.
Pulled out here to reduce the size of the catalog PR.
The edits include:
* Fix spacing
* Add type parameters to some types that omit them (to prevent the IDE from
flagging spurious warnings)
* Add `toString()` methods to formats so tests compare correctly. Without
this, an UT with an external data source will fail if the two formats are not
the same object. The change allows comparison as long as the two formats have
identical values.
* IT doc revisions
* Error message edits
* Display UT query results when tests fail. This makes it easy to copy/paste
the correct results into the test.
* When creating alert records, omit the `query` field if it is `null`.
(Else, the original error is masked by an NPE in the logging code.)
No substantial changes are in the PR.
#### Release note
No user-visible changes.
<hr>
This PR has:
- [X] been self-reviewed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org
For additional commands, e-mail: commits-h...@druid.apache.org
