[kudu] branch master updated: [subprocess] Fix subprocess JAR path

[granthenke]

This is an automated email from the ASF dual-hosted git repository.

granthenke pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git


The following commit(s) were added to refs/heads/master by this push:
     new 9c0a733  [subprocess] Fix subprocess JAR path
9c0a733 is described below

commit 9c0a7339465902afd25263edc3c8b27392141e49
Author: Attila Bukor <abu...@apache.org>
AuthorDate: Wed Mar 4 14:24:52 2020 +0100

    [subprocess] Fix subprocess JAR path
    
    f4176b1a was tested before c35871df was merged in. While they
    didn't conflict code-wise, the kudu-subprocess-echo.jar that f4176b1a
    tries to run in a test doesn't exist anymore since c35871df.
    
    This commit fixes the location of the JAR and provides the main class as
    it's not in the manifest of the JAR anymore.
    
    Change-Id: Icce4088580fcccfe34bddafb0d84f3835091f27c
    Reviewed-on: http://gerrit.cloudera.org:8080/15358
    Reviewed-by: Grant Henke <granthe...@apache.org>
    Tested-by: Kudu Jenkins
---
 src/kudu/subprocess/subprocess_proxy-test.cc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/kudu/subprocess/subprocess_proxy-test.cc 
b/src/kudu/subprocess/subprocess_proxy-test.cc
index 6d26268..958807e 100644
--- a/src/kudu/subprocess/subprocess_proxy-test.cc
+++ b/src/kudu/subprocess/subprocess_proxy-test.cc
@@ -114,7 +114,8 @@ class EchoSubprocessTest : public KuduTest {
     RETURN_NOT_OK(FindHomeDir("java", bin_dir, &java_home));
     vector<string> argv = {
       Substitute("$0/bin/java", java_home),
-      "-jar", Substitute("$0/kudu-subprocess-echo.jar", bin_dir)
+      "-cp", Substitute("$0/kudu-subprocess.jar", bin_dir),
+      "org.apache.kudu.subprocess.echo.EchoSubprocessMain"
     };
     echo_subprocess_ = make_shared<EchoSubprocess>(std::move(argv), 
metric_entity_);
     return echo_subprocess_->Start();