Skip to content
This repository has been archived by the owner on Aug 5, 2024. It is now read-only.

[fix] 'Loading: 0 packages loaded' message prohibition #71

Merged
merged 1 commit into from
Aug 3, 2021
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 16 additions & 1 deletion src/main/java/org/jetbrains/bsp/bazel/server/bep/BepServer.java
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,9 @@ public class BepServer extends PublishBuildEventGrpc.PublishBuildEventImplBase {

private static final Logger LOGGER = LogManager.getLogger(BepServer.class);

private static final List<String> EXCLUDED_PROGRESS_MESSAGES =
List.of("Loading: 0 packages loaded");

private static final int URI_PREFIX_LENGTH = 7;

private final BuildClient bspClient;
Expand Down Expand Up @@ -257,7 +260,7 @@ private void consumeProgressEvent(BuildEventStreamProtos.Progress progress) {
.map(this::createParamsFromEntry)
.forEach(bspClient::onBuildPublishDiagnostics);

buildClientLogger.logMessage(progress.getStderr().trim());
logProgress(progress);
}

private PublishDiagnosticsParams createParamsFromEntry(
Expand All @@ -272,6 +275,18 @@ private PublishDiagnosticsParams createParamsFromEntry(
false);
}

private void logProgress(BuildEventStreamProtos.Progress progress) {
String progressMessage = progress.getStderr().trim();

if (shouldMessageBeLogged(progressMessage)) {
buildClientLogger.logMessage(progressMessage);
}
}

private boolean shouldMessageBeLogged(String message) {
return EXCLUDED_PROGRESS_MESSAGES.stream().noneMatch(message::startsWith);
}

public Map<String, Set<Uri>> getOutputGroupPaths() {
return outputGroupPaths;
}
Expand Down