Lazy loading for s3_client and executor #41423

vnghia · 2020-07-15T16:23:40Z

@mihaimaruseac
This PR adds lazy-loading for s3_client and executor. This is the same as #41410 but I forgot to make GetExecutor static.

vnghia added 3 commits July 15, 2020 13:16

Add GetDefaultClientConfig

6d26d5d

Add get s3 client

fedf6fa

Add get executor

c4f258a

google-ml-butler bot added the size:M CL Change Size: Medium label Jul 15, 2020

google-ml-butler bot requested a review from mihaimaruseac July 15, 2020 16:23

googlebot added the cla: yes label Jul 15, 2020

vnghia force-pushed the s3-client branch from 4bbb367 to c4f258a Compare July 15, 2020 16:34

Pass s3_file instead of filesystem

11a1a56

goldiegadde assigned goldiegadde and unassigned goldiegadde Jul 15, 2020

mihaimaruseac approved these changes Jul 15, 2020

View reviewed changes

google-ml-butler bot added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels Jul 15, 2020

kokoro-team removed the kokoro:force-run Tests on submitted change label Jul 15, 2020

gbaned self-assigned this Jul 16, 2020

gbaned added this to Assigned Reviewer in PR Queue via automation Jul 16, 2020

tensorflow-copybara merged commit 5522bff into tensorflow:master Jul 16, 2020

PR Queue automation moved this from Assigned Reviewer to Merged Jul 16, 2020

vnghia deleted the s3-client branch July 16, 2020 14:56

Provide feedback