Skip to content

Commit

Permalink
Merge pull request #33120 from vespa-engine/magnus/windows-vespa-query
Browse files Browse the repository at this point in the history
fix: run vespa.exe directly on windows instead of unsanitized cmd.exe
  • Loading branch information
Mangern authored Jan 15, 2025
2 parents b9c6639 + 3d78940 commit 7133952
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,7 @@ private CompletableFuture<QueryResult> runVespaQuery(String query, ClientLogger
ProcessBuilder builder = new ProcessBuilder();

if (isWindows) {
builder.command("cmd.exe", "/c", "vespa", "query", query); // TODO: Test this on windows
builder.command("vespa.exe", "query", query);
} else {
builder.command("vespa", "query", query);
}
Expand Down

0 comments on commit 7133952

Please sign in to comment.