[
https://issues.apache.org/jira/browse/BEAM-10082?focusedWorklogId=447394&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-447394
]
ASF GitHub Bot logged work on BEAM-10082:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 17/Jun/20 16:42
Start Date: 17/Jun/20 16:42
Worklog Time Spent: 10m
Work Description: echauchot commented on a change in pull request #12001:
URL: https://github.com/apache/beam/pull/12001#discussion_r440982503
##########
File path:
runners/extensions-java/metrics/src/main/java/org/apache/beam/runners/extensions/metrics/MetricsHttpSink.java
##########
@@ -74,7 +74,9 @@ public void writeMetrics(MetricQueryResults
metricQueryResults) throws Exception
}
int responseCode = connection.getResponseCode();
if (responseCode != 200) {
- throw new IOException("Expected OK 200 response but received: " +
responseCode);
+ throw new IOException(
+ "Expected HTTP 200 OK response while writing metrics to
MetricsSinkHttp but received "
Review comment:
sed s/MetricsSinkHttp/MetricsHttpSink/
##########
File path:
runners/extensions-java/metrics/src/main/java/org/apache/beam/runners/extensions/metrics/MetricsHttpSink.java
##########
@@ -86,8 +93,8 @@ public MetricNameSerializer(Class<MetricName> t) {
public void serialize(MetricName value, JsonGenerator gen,
SerializerProvider provider)
throws IOException {
gen.writeStartObject();
- gen.writeObjectField("name", value.name());
- gen.writeObjectField("namespace", value.namespace());
Review comment:
yes I replaced them with get* versions to simplify serialization code
but they were re-introduced to be compatible with dataflow hosted engine which
uses .name and .namespace which I could not know of because it is not
opensource. Anyway good catch ! Dataflow is supposed to have migrated to .get*
versions now, can you check ? If so we can remove the deprecated versions from
the api.
----------------------------------------------------------------
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:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 447394)
Time Spent: 1h 20m (was: 1h 10m)
> Extensions Java Metric Tests failing [Java 11]
> ----------------------------------------------
>
> Key: BEAM-10082
> URL: https://issues.apache.org/jira/browse/BEAM-10082
> Project: Beam
> Issue Type: Sub-task
> Components: runner-extensions-metrics
> Reporter: Pawel Pasterz
> Assignee: Tyson Hamilton
> Priority: P2
> Time Spent: 1h 20m
> Remaining Estimate: 0h
>
> Gradle task *_:runners:extensions-java:metrics:test_* fails during Java 11
> Precommit job
>
> Example stack trace
> {code:java}
> > Task :runners:extensions-java:metrics:test FAILED
> Build cache key for task ':runners:extensions-java:metrics:test' is
> b5fe8f75576d2ea59a0fae51414b5c67
> Task ':runners:extensions-java:metrics:test' is not up-to-date because:
> No history is available.
> Custom actions are attached to task ':runners:extensions-java:metrics:test'.
> Starting process 'Gradle Test Executor 187'. Working directory:
> /home/jenkins/jenkins-slave/workspace/beam_PreCommit_Java11_Phrase/src/runners/extensions-java/metrics
> Command: /usr/lib/jvm/java-11-openjdk-amd64/bin/java
> -Dorg.gradle.native=false
> -javaagent:build/tmp/expandedArchives/org.jacoco.agent-0.8.2.jar_2aca8b620b19ecd063f63feff8caaa38/jacocoagent.jar=destfile=build/jacoco/test.exec,append=true,inclnolocationclasses=false,dumponexit=true,output=file,jmx=false
> @/tmp/gradle-worker-classpath3076868897892794496txt -Xmx2g
> -Dfile.encoding=UTF-8 -Duser.country=US -Duser.language=en -Duser.variant -ea
> worker.org.gradle.process.internal.worker.GradleWorkerMain 'Gradle Test
> Executor 187'
> Successfully started process 'Gradle Test Executor 187'
> org.apache.beam.runners.extensions.metrics.MetricsHttpSinkTest >
> testWriteMetricsWithCommittedUnSupported STANDARD_ERROR
> SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
> SLF4J: Defaulting to no-operation (NOP) logger implementation
> SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further
> details.
> org.apache.beam.runners.extensions.metrics.MetricsHttpSinkTest >
> testWriteMetricsWithCommittedUnSupported FAILED
> java.lang.NoClassDefFoundError: javax/xml/ws/http/HTTPException
> at
> org.apache.beam.runners.extensions.metrics.MetricsHttpSinkTest.testWriteMetricsWithCommittedUnSupported(MetricsHttpSinkTest.java:107)
> Caused by:
> java.lang.ClassNotFoundException: javax.xml.ws.http.HTTPException
> at
> jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:583)
> at
> jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178)
> at java.lang.ClassLoader.loadClass(ClassLoader.java:521)
> ... 1 more
> org.apache.beam.runners.extensions.metrics.MetricsHttpSinkTest >
> testWriteMetricsWithCommittedSupported FAILED
> java.lang.NoClassDefFoundError: javax/xml/ws/http/HTTPException
> at
> org.apache.beam.runners.extensions.metrics.MetricsHttpSinkTest.testWriteMetricsWithCommittedSupported(MetricsHttpSinkTest.java:85)
> Caused by:
> java.lang.ClassNotFoundException: javax.xml.ws.http.HTTPException
> at
> jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:583)
> at
> jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178)
> at java.lang.ClassLoader.loadClass(ClassLoader.java:521)
> ... 1 more
> 4 tests completed, 2 failed
> {code}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
