New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Catalog integration tests #16424
Catalog integration tests #16424
Conversation
if (sqlTaskStatus.getState().isFailure()) { | ||
Assert.fail(StringUtils.format( | ||
"Unable to start the task successfully.\nPossible exception: %s", | ||
sqlTaskStatus.getError() |
Check notice
Code scanning / CodeQL
Use of default toString() Note
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is a valid issue; could you implement toString
- or use jackson to flatten it down?
if (sqlTaskStatus.getState().isFailure()) { | ||
Assert.fail(StringUtils.format( | ||
"Unable to start the task successfully.\nPossible exception: %s", | ||
sqlTaskStatus.getError() |
Check notice
Code scanning / CodeQL
Use of default toString() Note
...sts-ex/cases/src/test/java/org/apache/druid/testsEx/catalog/ITCatalogIngestAndQueryTest.java
Dismissed
Show dismissed
Hide dismissed
if (sqlTaskStatus.getState().isFailure()) { | ||
Assert.fail(StringUtils.format( | ||
"Unable to start the task successfully.\nPossible exception: %s", | ||
sqlTaskStatus.getError() |
Check notice
Code scanning / CodeQL
Use of default toString() Note
...sts-ex/cases/src/test/java/org/apache/druid/testsEx/catalog/ITCatalogIngestAndQueryTest.java
Dismissed
Show dismissed
Hide dismissed
[ | ||
{ | ||
"query": "SELECT * FROM %%DATASOURCE%%", | ||
"expectedResults": [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just curious - how did you generated this?
if (sqlTaskStatus.getState().isFailure()) { | ||
Assert.fail(StringUtils.format( | ||
"Unable to start the task successfully.\nPossible exception: %s", | ||
sqlTaskStatus.getError() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is a valid issue; could you implement toString
- or use jackson to flatten it down?
Description
Add more Druid catalog integration tests.
This PR has: