hops-tensorflow icon indicating copy to clipboard operation
hops-tensorflow copied to clipboard

Client#main is broken

Open tobiajo opened this issue 8 years ago • 1 comments

Since this change:

   public Client() throws Exception {
 -    this(new YarnConfiguration());
 +    this.appMasterMainClass = ApplicationMaster.class.getName();
    }

Client#main is broken and therefore yarntf-submit too. The main method uses the empty constructor and now essentially lacks the following code:

    conf = new YarnConfiguration();
    yarnClient = YarnClient.createYarnClient();
    yarnClient.init(conf);
    yarnClient.start();
    opts = createOptions();

tobiajo avatar Jul 27 '17 13:07 tobiajo

I don't want to revert this with a PR, since I don't know how/if the empty constructor is used by Hopsworks.

tobiajo avatar Jul 27 '17 13:07 tobiajo