[
https://issues.apache.org/jira/browse/BEAM-14116?focusedWorklogId=744385&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-744385
]
ASF GitHub Bot logged work on BEAM-14116:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 18/Mar/22 20:04
Start Date: 18/Mar/22 20:04
Worklog Time Spent: 10m
Work Description: lukecwik commented on a change in pull request #17094:
URL: https://github.com/apache/beam/pull/17094#discussion_r830316956
##########
File path:
sdks/java/extensions/google-cloud-platform-core/src/main/java/org/apache/beam/sdk/extensions/gcp/util/GcsUtil.java
##########
@@ -466,7 +467,8 @@ SeekableByteChannel open(GcsPath path,
GoogleCloudStorageReadOptions readOptions
MonitoringInfoConstants.Labels.RESOURCE,
GcpResourceIdentifiers.cloudStorageBucket(path.getBucket()));
baseLabels.put(
- MonitoringInfoConstants.Labels.GCS_PROJECT_ID,
googleCloudStorageOptions.getProjectId());
+ MonitoringInfoConstants.Labels.GCS_PROJECT_ID,
+ MoreObjects.firstNonNull(googleCloudStorageOptions.getProjectId(),
""));
Review comment:
It looks like project id will not be set for the default case since the
GcsUtil does not set it at all.
I found
https://docs.google.com/document/d/1K_qsi1TMtt0fLxEjkM1taHIZX5V1csGAjh1pMALr4pI/edit#
but it does not seem to cover unknown values and what to do there. I did see
that in gcsio.py we try to get the project number so that would mean that we
would want to use String.valueOf(...) here which will convert `null` to
`"null"` similar to how unknown project numbers in Python will turn into
`"None"`. `"null"` will be more obvious that this isn't implemented.
Ditto for down below.
--
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]
Issue Time Tracking
-------------------
Worklog Id: (was: 744385)
Time Spent: 2h 20m (was: 2h 10m)
> Fix Pub/Sub Lite IO and SDF performance issues with shuffles
> ------------------------------------------------------------
>
> Key: BEAM-14116
> URL: https://issues.apache.org/jira/browse/BEAM-14116
> Project: Beam
> Issue Type: Task
> Components: io-java-gcp, runner-dataflow
> Reporter: Daniel Collins
> Assignee: Daniel Collins
> Priority: P2
> Time Spent: 2h 20m
> Remaining Estimate: 0h
>
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
