[GitHub] [beam] pabloem commented on a change in pull request #14024: Add null checks to FhirIO.Search for when the search results are empty.

[GitBox <gi...@apache.org>]

pabloem commented on a change in pull request #14024:
URL: https://github.com/apache/beam/pull/14024#discussion_r579449042



##########
File path: sdks/java/io/google-cloud-platform/src/test/java/org/apache/beam/sdk/io/gcp/healthcare/FhirIOSearchIT.java
##########
@@ -21,6 +21,7 @@
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotEquals;
 
+import com.google.common.collect.ImmutableMap;

Review comment:
       In Beam we use a vendored version of Guava. This is what's causing the precommit failure.
   
   You'll need to import our vendored guava (and also reorder de import alphabetically I guess)
   ```suggestion
   import org.apache.beam.vendor.guava.v26_0_jre.com.google.common.collect.ImmutableMap;
   ```




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org