FrankChen021 commented on pull request #12026: URL: https://github.com/apache/druid/pull/12026#issuecomment-991047526
@jihoonson This patch updates many returning statements, it's hard to add cases for all these changes. For the tests: I added a UT test case to check the returned `Response` object, including the Header, Object and Status Code; can check this file: indexing-service/src/test/java/org/apache/druid/indexing/overlord/http/OverlordResourceTest.java I also updated current integration test to check if the `ResponseStatusExceptionMapper` successfully coverts exception into `Response` object and returned Header, Content and Status are correct. See: integration-tests/src/test/java/org/apache/druid/tests/query/ITOverlordResourceTest.java -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
