aldettinger commented on code in PR #4751:
URL: https://github.com/apache/camel-quarkus/pull/4751#discussion_r1159875497
##########
integration-tests/jdbc/src/main/java/org/apache/camel/quarkus/component/jdbc/CamelResource.java:
##########
@@ -88,22 +87,22 @@ public String getSpeciesById(@PathParam("id") String id)
throws Exception {
@Produces(MediaType.TEXT_PLAIN)
public String getSpeciesByIdWithSelectList(@PathParam("id") String id)
throws Exception {
List<LinkedHashMap<String, Object>> result = template
- .requestBody("jdbc:camel-ds?outputType=SelectList", "select *
from camels where id = " + id, List.class);
+ .requestBody("jdbc:cameldb?outputType=SelectList", "select *
from camels where id = " + id, List.class);
if (result.isEmpty()) {
throw new IllegalStateException("Expected at least 1 camel result
but none were found");
}
LinkedHashMap<String, Object> data = result.get(0);
- return data.get("SPECIES") + " " + data.get("ID");
+ return data.toString();
Review Comment:
Is it really better to use toString() ? What happen if the toString()
implementation changes ?
--
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...@camel.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
