MINOR: [Java][Flight] Add context of descriptor on integration tests when not found

raulcd · lidavidm · commit 26f2d877b5d0 · 2022-05-03T12:03:29.000-04:00
This PR adds minor logging info about the descriptor looked up when it can't be found. For example on this ticket failure [ARROW-16441](https://issues.apache.org/jira/browse/ARROW-16441) we should be able to get more info around the descriptor: ``` panic: rpc error: code = NotFound desc = Unknown descriptor. ``` Closes #13053 from raulcd/add-descriptor-info-to-not-found Authored-by: Raúl Cumplido <raulcumplido@gmail.com> Signed-off-by: David Li <li.davidm96@gmail.com>
diff --git a/java/flight/flight-integration-tests/src/main/java/org/apache/arrow/flight/integration/tests/IntegrationProducer.java b/java/flight/flight-integration-tests/src/main/java/org/apache/arrow/flight/integration/tests/IntegrationProducer.java
@@ -97,7 +97,7 @@ public void getStream(CallContext context, Ticket ticket, ServerStreamListener l
   public FlightInfo getFlightInfo(CallContext context, FlightDescriptor descriptor) {
     Dataset h = datasets.get(descriptor);
     if (h == null) {
-      throw CallStatus.NOT_FOUND.withDescription("Unknown descriptor.").toRuntimeException();
+      throw CallStatus.NOT_FOUND.withDescription("Unknown descriptor: " + descriptor).toRuntimeException();
     }
     return h.getFlightInfo(location);
   }

Original file line number	Diff line number	Diff line change
`@@ -97,7 +97,7 @@ public void getStream(CallContext context, Ticket ticket, ServerStreamListener l`
`97`	`97`	`public FlightInfo getFlightInfo(CallContext context, FlightDescriptor descriptor) {`
`98`	`98`	`Dataset h = datasets.get(descriptor);`
`99`	`99`	`if (h == null) {`
`100`		`- throw CallStatus.NOT_FOUND.withDescription("Unknown descriptor.").toRuntimeException();`
	`100`	`+ throw CallStatus.NOT_FOUND.withDescription("Unknown descriptor: " + descriptor).toRuntimeException();`
`101`	`101`	`}`
`102`	`102`	`return h.getFlightInfo(location);`
`103`	`103`	`}`