Kyle Weaver created BEAM-7460:
---------------------------------

             Summary: Refactor ExecutorServiceFactory out of GcsOptions
                 Key: BEAM-7460
                 URL: https://issues.apache.org/jira/browse/BEAM-7460
             Project: Beam
          Issue Type: Improvement
          Components: sdk-java-harness
            Reporter: Kyle Weaver


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]

[2] 
[https://github.com/apache/beam/blob/39dab79b7bd56a47c0f2a02033ab4ca3bd4a67e2/sdks/java/harness/src/main/java/org/apache/beam/fn/harness/FnHarness.java#L170]



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to