andoni-guzman commented on a change in pull request #17046:
URL: https://github.com/apache/beam/pull/17046#discussion_r832517755
##########
File path: sdks/go/pkg/beam/core/runtime/graphx/translate.go
##########
@@ -36,7 +36,7 @@ import (
// TODO(lostluck): 2018/05/28 Extract these from their enum descriptors in the
pipeline_v1 proto
const (
URNImpulse = "beam:transform:impulse:v1"
- URNParDo = "beam:transform:pardo:v1"
+ URNParDo = "beam:transform:org.apache.beam:pardo:v1"
Review comment:
Hi @ibzib I disable the tests that failed and back to the original URN.
I think this PR is ready to merge.
A question related with the tracking of the disabled tests. It's better
create new jiras for each error ( at this moment there is two error that affect
the tests) and close BEAM-12898 Leaking Vm's or still had open beam 12898 with
a note in the comments that dataproc creation,deletion and fixed vm's is solved
and only the tests are missing?
--
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]
