mosche commented on code in PR #22545:
URL: https://github.com/apache/beam/pull/22545#discussion_r942150452
##########
sdks/java/testing/test-utils/src/main/java/org/apache/beam/sdk/testutils/publishing/InfluxDBPublisher.java:
##########
@@ -64,6 +64,13 @@ public static void publishNexmarkResults(
publishWithCheck(settings, () -> publishNexmark(results, settings, tags));
}
+ public static void publishTpcdsResults(
Review Comment:
https://github.com/apache/beam/pull/22260 requires a review, I can ping Luke
to see if he's back or if you wanna do it that would be perfect as well...
Regarding one API .... IMHO I don't think that should be the goal, I'd
consider it to be much more important to not build new things on clearly broken
and poor old abstractions / code
--
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]
