kennknowles opened a new issue, #19486:
URL: https://github.com/apache/beam/issues/19486

   The SDK harness uses GcsOptions.ExecutorServiceFactory  [1] to process 
instruction requests [2]. I found this dependency really surprising, and it 
also happens to be unnecessary afaict, so ExecutorServiceFactory should 
probably get factored out into its own class.
   
   [1] 
[https://github.com/apache/beam/blob/39dab79b7bd56a47c0f2a02033ab4ca3bd4a67e2/sdks/java/extensions/google-cloud-platform-core/src/main/java/org/apache/beam/sdk/extensions/gcp/options/GcsOptions.java#L136-L158](https://github.com/apache/beam/blob/39dab79b7bd56a47c0f2a02033ab4ca3bd4a67e2/sdks/java/extensions/google-cloud-platform-core/src/main/java/org/apache/beam/sdk/extensions/gcp/options/GcsOptions.java#L136-L158)
   
   [2] 
[https://github.com/apache/beam/blob/39dab79b7bd56a47c0f2a02033ab4ca3bd4a67e2/sdks/java/harness/src/main/java/org/apache/beam/fn/harness/FnHarness.java#L170](https://github.com/apache/beam/blob/39dab79b7bd56a47c0f2a02033ab4ca3bd4a67e2/sdks/java/harness/src/main/java/org/apache/beam/fn/harness/FnHarness.java#L170)
   
   Imported from Jira 
[BEAM-7460](https://issues.apache.org/jira/browse/BEAM-7460). Original Jira may 
contain additional context.
   Reported by: ibzib.


-- 
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]

Reply via email to