Skip to content

Commit

Permalink
Added start_time and temp_space fields to JobDescription message
Browse files Browse the repository at this point in the history
Fixes #43
  • Loading branch information
sverhoeven committed Jun 14, 2019
1 parent a8ca95f commit 5f92022
Show file tree
Hide file tree
Showing 4 changed files with 29 additions and 0 deletions.
17 changes: 17 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,23 @@ and this project adheres to [Semantic Versioning](http://semver.org/).

## Unreleased

## [3.0.0] - 2019-06-14

### Added

* start_time and temp_space fields to JobDescription message ([#43](https://github.com/xenon-middleware/xenon-grpc/issues/43))
* [at](https://linux.die.net/man/1/at) scheduler

### Changed

* Replaced tasks+cores+nodes fields in JobDescription message with nodes+processes+thread fields ([#625](https://github.com/xenon-middleware/xenon/issues/625)).
* Require Java 11 or greater, as xenon package has same compatibility
* Upgraded to Xenon 3.0.0

### Removed

* hdfs filesystem

## [2018-03-14] 2.3.0

### Added
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -97,6 +97,12 @@ public static JobDescription mapJobDescription(XenonProto.JobDescription d) {
if (d.getSchedulerArgumentsCount() > 0) {
description.setSchedulerArguments(d.getSchedulerArgumentsList().toArray(new String[0]));
}
if (!d.getStartTime().equals("")) {
description.setStartTime(d.getStartTime());
}
if (d.getTempSpace() != 0) {
description.setTempSpace(d.getTempSpace());
}
return description;
}

Expand Down
2 changes: 2 additions & 0 deletions src/main/proto/xenon.proto
Original file line number Diff line number Diff line change
Expand Up @@ -346,6 +346,8 @@ message JobDescription {
uint32 tasks_per_node = 19;
// In Xenon if true then uses setStartPerTask else uses setStartPerJob to configure how to start executable
bool start_per_task = 20;
string start_time = 21;
uint32 temp_space = 22;
}

message SubmitBatchJobRequest {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,8 @@ public void custom() {
.setName("myjobname")
.setMaxMemory(4096)
.addAllSchedulerArguments(schedulerArgs)
.setStartTime("now")
.setTempSpace(512)
;

XenonProto.JobDescription request = builder.build();
Expand All @@ -75,6 +77,8 @@ public void custom() {
expected.setName("myjobname");
expected.setMaxMemory(4096);
expected.setSchedulerArguments("schedarg1", "schedarg2");
expected.setStartTime("now");
expected.setTempSpace(512);
assertEquals(expected, response);
assertEquals(schedulerArgs, response.getSchedulerArguments());
}
Expand Down

0 comments on commit 5f92022

Please sign in to comment.