diff options
author | tomlu <tomlu@google.com> | 2018-07-04 08:55:42 -0700 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-07-04 08:57:40 -0700 |
commit | 2348a75882bda4ed8b983fd4f8af2c45fa292e10 (patch) | |
tree | 50e6b82eb7c4d7ba1556a7696e266b630082e87f /src/main/java/com/google/devtools/build/lib/buildeventstream/BuildEventArtifactUploader.java | |
parent | 9b6208c6d74a5d485bed43f4a214f331e32d8ef6 (diff) |
Include more information about build events' referenced local files.
Instead of just a path, events now include information about the type of file (output, source file, stdout/stderr, test logs, etc.). This information can be used by the uploaders to determine a) whether to upload, b) what kind of lease to give the files.
RELNOTES: None
PiperOrigin-RevId: 203285549
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/buildeventstream/BuildEventArtifactUploader.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/buildeventstream/BuildEventArtifactUploader.java | 28 |
1 files changed, 15 insertions, 13 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/buildeventstream/BuildEventArtifactUploader.java b/src/main/java/com/google/devtools/build/lib/buildeventstream/BuildEventArtifactUploader.java index 5c1fe9c5b2..3f536325db 100644 --- a/src/main/java/com/google/devtools/build/lib/buildeventstream/BuildEventArtifactUploader.java +++ b/src/main/java/com/google/devtools/build/lib/buildeventstream/BuildEventArtifactUploader.java @@ -15,30 +15,32 @@ package com.google.devtools.build.lib.buildeventstream; import com.google.common.util.concurrent.Futures; import com.google.common.util.concurrent.ListenableFuture; +import com.google.devtools.build.lib.buildeventstream.BuildEvent.LocalFile; import com.google.devtools.build.lib.buildeventstream.PathConverter.FileUriPathConverter; import com.google.devtools.build.lib.vfs.Path; -import java.util.Set; +import java.util.Map; /** Uploads artifacts referenced by the Build Event Protocol (BEP). */ public interface BuildEventArtifactUploader { - BuildEventArtifactUploader LOCAL_FILES_UPLOADER = new BuildEventArtifactUploader() { - private final ListenableFuture<PathConverter> completedPathConverter = - Futures.immediateFuture(new FileUriPathConverter()); + BuildEventArtifactUploader LOCAL_FILES_UPLOADER = + new BuildEventArtifactUploader() { + private final ListenableFuture<PathConverter> completedPathConverter = + Futures.immediateFuture(new FileUriPathConverter()); - @Override - public ListenableFuture<PathConverter> upload(Set<Path> files) { - return completedPathConverter; - } - }; + @Override + public ListenableFuture<PathConverter> upload(Map<Path, LocalFile> files) { + return completedPathConverter; + } + }; /** - * Asynchronously uploads a set of files referenced by the protobuf representation of a - * {@link BuildEvent}. This method is expected to return quickly. + * Asynchronously uploads a set of files referenced by the protobuf representation of a {@link + * BuildEvent}. This method is expected to return quickly. * * <p>This method must not throw any exceptions. - * + * * <p>Returns a future to a {@link PathConverter} that must provide a name for each uploaded file * as it should appear in the BEP. */ - ListenableFuture<PathConverter> upload(Set<Path> files); + ListenableFuture<PathConverter> upload(Map<Path, LocalFile> files); } |