Increase Jaeger Tracing coverage (feast-dev#719)

terryyylim · web-flow · commit a33883fbbf0c · 2020-06-01T16:11:59.000+08:00
* Add jaegertracing logs to getOnlineFeatures

* Apply spotless
diff --git a/serving/src/main/java/feast/serving/service/OnlineServingService.java b/serving/src/main/java/feast/serving/service/OnlineServingService.java
@@ -16,6 +16,7 @@
  */
 package feast.serving.service;
 
+import com.google.common.collect.ImmutableMap;
 import com.google.common.collect.Maps;
 import com.google.protobuf.Duration;
 import feast.proto.serving.ServingAPIProto.*;
@@ -75,6 +76,9 @@ public GetOnlineFeaturesResponse getOnlineFeatures(GetOnlineFeaturesRequest requ
       // feature set request.
       List<List<FeatureRow>> featureRows =
           retriever.getOnlineFeatures(entityRows, featureSetRequests);
+      if (scope != null) {
+        scope.span().log(ImmutableMap.of("event", "featureRows", "value", featureRows));
+      }
 
       // For each feature set request, read the feature rows returned by the retriever, and
       // populate the featureValuesMap with the feature values corresponding to that entity row.
