kamilwu commented on a change in pull request #13272:
URL: https://github.com/apache/beam/pull/13272#discussion_r520607900
##########
File path: sdks/go/pkg/beam/core/monitoring/monitoring.go
##########
@@ -13,10 +13,12 @@
// See the License for the specific language governing permissions and
// limitations under the License.
-package harness
+package monitoring
Review comment:
I created the new metricsx package which exports payload helpers and urn
constants (which are required both by harness and my changes). I don't like
copying over 100 lines of code, so the package seems like a good idea.
----------------------------------------------------------------
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]