Re: Review Request 40149: Create ExecutorSettings creation closer to command line arguments.

2015-11-11 Thread Bill Farner 


> On Nov. 10, 2015, 2:07 p.m., Zameer Manji wrote:
> > src/main/java/org/apache/aurora/scheduler/app/SchedulerMain.java, line 227
> > 
> >
> > Is there a reason why we are not using Java's URI type to do this sort 
> > of parsing?

First - i'd like to point out that this is not new code (it came from the 
now-deleted `CommandUtil`).  As for using URI, i'd like to play agnostic in 
this patch and not impose new format rules by using `URI` parsing.


- Bill


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40149/#review105966
---


On Nov. 10, 2015, 12:42 p.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/40149/
> ---
> 
> (Updated Nov. 10, 2015, 12:42 p.m.)
> 
> 
> Review request for Aurora and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> This is prep work for https://issues.apache.org/jira/browse/AURORA-1376
> 
> The high-level goal is to push ExecutorInfo creation closer to command line 
> args so they can be fully specified there in subsequent changes.  A lot of 
> the changes here are code shuffle, but there is one behavior change in 
> `MesosTaskFactory`.  There is currently a dance where `RESOURCES_EPSILON` is 
> used to effectively account all resources under the task.  With this change, 
> the executor resources are used verbatim and kept distinct from the task's 
> resources.
> 
> 
> Diffs
> -
> 
>   config/legacy_untested_classes.txt b77265ca1c89f9a2d7e6e01dea9f04b200e4ef6b 
>   src/jmh/java/org/apache/aurora/benchmark/SchedulingBenchmarks.java 
> c210c0db07bb1f4b3f76668178dcd7e2de56a4ac 
>   src/jmh/java/org/apache/aurora/benchmark/StatusUpdateBenchmark.java 
> 197184b6edc0768d677636341b5737f262abdf7d 
>   src/main/java/org/apache/aurora/scheduler/ResourceSlot.java 
> c665c3237a03f0fadf8ca158b97cad14dd1eea81 
>   src/main/java/org/apache/aurora/scheduler/app/SchedulerMain.java 
> 99b874459c358c67234c6fe748e6427f313a04a8 
>   src/main/java/org/apache/aurora/scheduler/base/CommandUtil.java 
> aa5ce8b2f14c7dbd0eae120018ee41387c26059f 
>   src/main/java/org/apache/aurora/scheduler/filter/SchedulingFilterImpl.java 
> fb0f7ccf7f4dfb3ccb2a4cb4a5643c4b5487ff8f 
>   src/main/java/org/apache/aurora/scheduler/mesos/ExecutorConfig.java 
> PRE-CREATION 
>   src/main/java/org/apache/aurora/scheduler/mesos/ExecutorSettings.java 
> b3c913892248e4a9a8111412307463985f5ca97f 
>   src/main/java/org/apache/aurora/scheduler/mesos/Executors.java PRE-CREATION 
>   src/main/java/org/apache/aurora/scheduler/mesos/MesosTaskFactory.java 
> f6ba2c40aea555d3e0ab774218bfe08d7e1c984b 
>   src/main/java/org/apache/aurora/scheduler/mesos/TestExecutorSettings.java 
> PRE-CREATION 
>   
> src/main/java/org/apache/aurora/scheduler/preemptor/PreemptionVictimFilter.java
>  67d7f072de045d4211b6239119802473278d6419 
>   src/test/java/org/apache/aurora/scheduler/ResourceSlotTest.java 
> 6fad3344042dc6a75cdf74ce79d388fcd4fc9861 
>   src/test/java/org/apache/aurora/scheduler/app/SchedulerIT.java 
> f63d6f12d114c062f81137ffc0b1078837e3cf76 
>   src/test/java/org/apache/aurora/scheduler/base/CommandUtilTest.java 
> cd0295780d41bc4e914583f195b37eaed28a46dc 
>   
> src/test/java/org/apache/aurora/scheduler/events/NotifyingSchedulingFilterTest.java
>  0576704f6bcdc2a7568ee1597613f71f33e092d9 
>   
> src/test/java/org/apache/aurora/scheduler/mesos/MesosTaskFactoryImplTest.java 
> dddf7952d3f0e508cd736d5fb95e573267708d43 
>   src/test/java/org/apache/aurora/scheduler/mesos/TaskExecutors.java 
> d0987251b058988fcbfab16c1b138c37e0c5b8c6 
>   
> src/test/java/org/apache/aurora/scheduler/preemptor/PreemptionVictimFilterTest.java
>  8ce8da88fe705bfb9846a790f76bfeb6b4af2a06 
> 
> Diff: https://reviews.apache.org/r/40149/diff/
> 
> 
> Testing
> ---
> 
> End-to-end tests pass.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 40149: Create ExecutorSettings closer to command line arguments.

2015-11-11 Thread Bill Farner 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40149/
---

(Updated Nov. 11, 2015, 8:16 a.m.)


Review request for Aurora and Zameer Manji.


Summary (updated)
-

Create ExecutorSettings closer to command line arguments.


Repository: aurora


Description
---

This is prep work for https://issues.apache.org/jira/browse/AURORA-1376

The high-level goal is to push ExecutorInfo creation closer to command line 
args so they can be fully specified there in subsequent changes.  A lot of the 
changes here are code shuffle, but there is one behavior change in 
`MesosTaskFactory`.  There is currently a dance where `RESOURCES_EPSILON` is 
used to effectively account all resources under the task.  With this change, 
the executor resources are used verbatim and kept distinct from the task's 
resources.


Diffs
-

  config/legacy_untested_classes.txt b77265ca1c89f9a2d7e6e01dea9f04b200e4ef6b 
  src/jmh/java/org/apache/aurora/benchmark/SchedulingBenchmarks.java 
c210c0db07bb1f4b3f76668178dcd7e2de56a4ac 
  src/jmh/java/org/apache/aurora/benchmark/StatusUpdateBenchmark.java 
197184b6edc0768d677636341b5737f262abdf7d 
  src/main/java/org/apache/aurora/scheduler/ResourceSlot.java 
c665c3237a03f0fadf8ca158b97cad14dd1eea81 
  src/main/java/org/apache/aurora/scheduler/app/SchedulerMain.java 
99b874459c358c67234c6fe748e6427f313a04a8 
  src/main/java/org/apache/aurora/scheduler/base/CommandUtil.java 
aa5ce8b2f14c7dbd0eae120018ee41387c26059f 
  src/main/java/org/apache/aurora/scheduler/filter/SchedulingFilterImpl.java 
fb0f7ccf7f4dfb3ccb2a4cb4a5643c4b5487ff8f 
  src/main/java/org/apache/aurora/scheduler/mesos/ExecutorConfig.java 
PRE-CREATION 
  src/main/java/org/apache/aurora/scheduler/mesos/ExecutorSettings.java 
b3c913892248e4a9a8111412307463985f5ca97f 
  src/main/java/org/apache/aurora/scheduler/mesos/Executors.java PRE-CREATION 
  src/main/java/org/apache/aurora/scheduler/mesos/MesosTaskFactory.java 
f6ba2c40aea555d3e0ab774218bfe08d7e1c984b 
  src/main/java/org/apache/aurora/scheduler/mesos/TestExecutorSettings.java 
PRE-CREATION 
  
src/main/java/org/apache/aurora/scheduler/preemptor/PreemptionVictimFilter.java 
67d7f072de045d4211b6239119802473278d6419 
  src/test/java/org/apache/aurora/scheduler/ResourceSlotTest.java 
6fad3344042dc6a75cdf74ce79d388fcd4fc9861 
  src/test/java/org/apache/aurora/scheduler/app/SchedulerIT.java 
f63d6f12d114c062f81137ffc0b1078837e3cf76 
  src/test/java/org/apache/aurora/scheduler/base/CommandUtilTest.java 
cd0295780d41bc4e914583f195b37eaed28a46dc 
  
src/test/java/org/apache/aurora/scheduler/events/NotifyingSchedulingFilterTest.java
 0576704f6bcdc2a7568ee1597613f71f33e092d9 
  src/test/java/org/apache/aurora/scheduler/mesos/MesosTaskFactoryImplTest.java 
dddf7952d3f0e508cd736d5fb95e573267708d43 
  src/test/java/org/apache/aurora/scheduler/mesos/TaskExecutors.java 
d0987251b058988fcbfab16c1b138c37e0c5b8c6 
  
src/test/java/org/apache/aurora/scheduler/preemptor/PreemptionVictimFilterTest.java
 8ce8da88fe705bfb9846a790f76bfeb6b4af2a06 

Diff: https://reviews.apache.org/r/40149/diff/


Testing
---

End-to-end tests pass.


Thanks,

Bill Farner

Re: Review Request 40161: Clarify the commit process for new contributors.

2015-11-11 Thread Bill Farner 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40161/#review106081
---

Ship it!


Ship It!

- Bill Farner


On Nov. 10, 2015, 3:40 p.m., John Sirois wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/40161/
> ---
> 
> (Updated Nov. 10, 2015, 3:40 p.m.)
> 
> 
> Review request for Aurora, Bill Farner and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> A slight emphasis change on the 'Merging' section that favors the new
> contributor and makes their path to master more front and center.
> 
> 
> Diffs
> -
> 
>   CONTRIBUTING.md 5cdb4b1115895d7f1195b4a7135d792003a29a68 
> 
> Diff: https://reviews.apache.org/r/40161/diff/
> 
> 
> Testing
> ---
> 
> Eyeballed:
> `markdown_py CONTRIBUTING.md > CONTRIBUTING.html && open CONTRIBUTING.html`
> 
> 
> Thanks,
> 
> John Sirois
> 
>

Re: Review Request 40212: Trim whitespace for pants version

2015-11-11 Thread Aurora ReviewBot 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40212/#review106181
---

Ship it!


Master (cf38611) is green with this patch.
  ./build-support/jenkins/build.sh

I will refresh this build result if you post a review containing "@ReviewBot 
retry"

- Aurora ReviewBot


On Nov. 12, 2015, 12:30 a.m., Joe Smith wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/40212/
> ---
> 
> (Updated Nov. 12, 2015, 12:30 a.m.)
> 
> 
> Review request for Aurora and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Trim whitespace for pants version
> 
> Contains https://github.com/pantsbuild/setup/pull/9
> 
> 
> Diffs
> -
> 
>   pants 4239bee415c580c57e68cd8db3508172fe7fe92a 
> 
> Diff: https://reviews.apache.org/r/40212/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Joe Smith
> 
>

Review Request 40204: Update pants bootstrap script to be agnostic to sed version

2015-11-11 Thread Joe Smith 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40204/
---

Review request for Aurora.


Repository: aurora


Description
---

Update pants bootstrap script to be agnostic to sed version


Diffs
-

  pants 47097994b1044202aa0d8ce6afb8c2dee2a4c27c 

Diff: https://reviews.apache.org/r/40204/diff/


Testing
---


Thanks,

Joe Smith

Re: Review Request 40204: Update pants bootstrap script to be agnostic to sed version

2015-11-11 Thread John Sirois 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40204/#review106171
---

Ship it!


Ship It!

- John Sirois


On Nov. 11, 2015, 4:39 p.m., Joe Smith wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/40204/
> ---
> 
> (Updated Nov. 11, 2015, 4:39 p.m.)
> 
> 
> Review request for Aurora, John Sirois and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Update pants bootstrap script to be agnostic to sed version
> 
> 
> Diffs
> -
> 
>   pants 47097994b1044202aa0d8ce6afb8c2dee2a4c27c 
> 
> Diff: https://reviews.apache.org/r/40204/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Joe Smith
> 
>

Re: Review Request 40204: Update pants bootstrap script to be agnostic to sed version

2015-11-11 Thread Zameer Manji 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40204/#review106172
---

Ship it!


Ship It!

- Zameer Manji


On Nov. 11, 2015, 3:39 p.m., Joe Smith wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/40204/
> ---
> 
> (Updated Nov. 11, 2015, 3:39 p.m.)
> 
> 
> Review request for Aurora, John Sirois and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Update pants bootstrap script to be agnostic to sed version
> 
> 
> Diffs
> -
> 
>   pants 47097994b1044202aa0d8ce6afb8c2dee2a4c27c 
> 
> Diff: https://reviews.apache.org/r/40204/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Joe Smith
> 
>

Re: Review Request 40204: Update pants bootstrap script to be agnostic to sed version

2015-11-11 Thread Aurora ReviewBot 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40204/#review106173
---

Ship it!


Master (0b025a7) is green with this patch.
  ./build-support/jenkins/build.sh

I will refresh this build result if you post a review containing "@ReviewBot 
retry"

- Aurora ReviewBot


On Nov. 11, 2015, 11:39 p.m., Joe Smith wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/40204/
> ---
> 
> (Updated Nov. 11, 2015, 11:39 p.m.)
> 
> 
> Review request for Aurora, John Sirois and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Update pants bootstrap script to be agnostic to sed version
> 
> 
> Diffs
> -
> 
>   pants 47097994b1044202aa0d8ce6afb8c2dee2a4c27c 
> 
> Diff: https://reviews.apache.org/r/40204/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Joe Smith
> 
>

Review Request 40208: Eliminate OOB pip install of python deps in CI.

2015-11-11 Thread John Sirois 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40208/
---

Review request for Aurora, Bill Farner and Zameer Manji.


Bugs: AURORA-954
https://issues.apache.org/jira/browse/AURORA-954


Repository: aurora


Description
---

Under older pants, and thus older pex, distributions were fetched using
urllib and were not robust to flaky connections. Now that Aurora is on
modern pants and pex, which uses the `requests` library for fetching and
includes 5 retries by default, this step should no longer be needed to
ensure stable CI runs.

 build-support/jenkins/build.sh | 7 ---
 pants.ini  | 5 -
 2 files changed, 12 deletions(-)


Diffs
-

  build-support/jenkins/build.sh 5cd5242a2e4551e356e5ce5d15a0d27beb7e2d4e 
  pants.ini 0bd8ec829ae65e2296a122166dea13f315323c9f 

Diff: https://reviews.apache.org/r/40208/diff/


Testing
---

I still can't quite run `./build-support/jenkins/build.sh` straight up
due to https://issues.apache.org/jira/browse/AURORA-1083 but ran the
following successfully locally which forces re-download of requirements
by pants:
```
$ ./pants clean-all test.pytest --no-fast src/test/python::
```


Thanks,

John Sirois

Review Request 40197: Fix `./pants test src/test/python::` to work out of the box.

2015-11-11 Thread John Sirois 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40197/
---

Review request for Aurora, Joshua Cohen and Bill Farner.


Bugs: AURORA-547
https://issues.apache.org/jira/browse/AURORA-547


Repository: aurora


Description
---

The Aurora python tests cannot currently all be run together; so the ci
script passes --no-fast and devs need to remember to do this too.
Improve the dev experience by defaulting this option.

 build-support/jenkins/build.sh | 2 +-
 pants.ini  | 7 +++
 2 files changed, 8 insertions(+), 1 deletion(-)


Diffs
-

  build-support/jenkins/build.sh 5cd5242a2e4551e356e5ce5d15a0d27beb7e2d4e 
  pants.ini 0bd8ec829ae65e2296a122166dea13f315323c9f 

Diff: https://reviews.apache.org/r/40197/diff/


Testing
---

Ran into this working https://issues.apache.org/jira/browse/AURORA-547.
After the fix, locally ran `./pants test src/test/python:: -- -v` green.
Previously this would lead to test failures and CPU starvation.


Thanks,

John Sirois

Re: Review Request 40197: Fix `./pants test src/test/python::` to work out of the box.

2015-11-11 Thread Bill Farner 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40197/#review106149
---

Ship it!


Ship It!

- Bill Farner


On Nov. 11, 2015, 12:30 p.m., John Sirois wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/40197/
> ---
> 
> (Updated Nov. 11, 2015, 12:30 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen and Bill Farner.
> 
> 
> Bugs: AURORA-547
> https://issues.apache.org/jira/browse/AURORA-547
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> The Aurora python tests cannot currently all be run together; so the ci
> script passes --no-fast and devs need to remember to do this too.
> Improve the dev experience by defaulting this option.
> 
>  build-support/jenkins/build.sh | 2 +-
>  pants.ini  | 7 +++
>  2 files changed, 8 insertions(+), 1 deletion(-)
> 
> 
> Diffs
> -
> 
>   build-support/jenkins/build.sh 5cd5242a2e4551e356e5ce5d15a0d27beb7e2d4e 
>   pants.ini 0bd8ec829ae65e2296a122166dea13f315323c9f 
> 
> Diff: https://reviews.apache.org/r/40197/diff/
> 
> 
> Testing
> ---
> 
> Ran into this working https://issues.apache.org/jira/browse/AURORA-547.
> After the fix, locally ran `./pants test src/test/python:: -- -v` green.
> Previously this would lead to test failures and CPU starvation.
> 
> 
> Thanks,
> 
> John Sirois
> 
>

Re: Review Request 40197: Fix `./pants test src/test/python::` to work out of the box.

2015-11-11 Thread Aurora ReviewBot 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40197/#review106147
---

Ship it!


Master (0b025a7) is green with this patch.
  ./build-support/jenkins/build.sh

I will refresh this build result if you post a review containing "@ReviewBot 
retry"

- Aurora ReviewBot


On Nov. 11, 2015, 8:30 p.m., John Sirois wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/40197/
> ---
> 
> (Updated Nov. 11, 2015, 8:30 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen and Bill Farner.
> 
> 
> Bugs: AURORA-547
> https://issues.apache.org/jira/browse/AURORA-547
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> The Aurora python tests cannot currently all be run together; so the ci
> script passes --no-fast and devs need to remember to do this too.
> Improve the dev experience by defaulting this option.
> 
>  build-support/jenkins/build.sh | 2 +-
>  pants.ini  | 7 +++
>  2 files changed, 8 insertions(+), 1 deletion(-)
> 
> 
> Diffs
> -
> 
>   build-support/jenkins/build.sh 5cd5242a2e4551e356e5ce5d15a0d27beb7e2d4e 
>   pants.ini 0bd8ec829ae65e2296a122166dea13f315323c9f 
> 
> Diff: https://reviews.apache.org/r/40197/diff/
> 
> 
> Testing
> ---
> 
> Ran into this working https://issues.apache.org/jira/browse/AURORA-547.
> After the fix, locally ran `./pants test src/test/python:: -- -v` green.
> Previously this would lead to test failures and CPU starvation.
> 
> 
> Thanks,
> 
> John Sirois
> 
>

Review Request 40201: Cleanup thermos_executor test pexes.

2015-11-11 Thread John Sirois 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40201/
---

Review request for Aurora, Kevin Sweeney, Brian Wickman, and Zameer Manji.


Bugs: AURORA-547
https://issues.apache.org/jira/browse/AURORA-547


Repository: aurora


Description
---

Previously these were built to the standard `dist/` dir, now they are
dumped to a tmp dir that's cleaned up.

 src/test/python/apache/aurora/executor/test_thermos_executor.py| 24 
+++-
 src/test/python/apache/aurora/executor/test_thermos_task_runner.py | 15 
++-
 2 files changed, 29 insertions(+), 10 deletions(-)


Diffs
-

  src/test/python/apache/aurora/executor/test_thermos_executor.py 
d24c61162e11bb4afef2902ebb031478fe3ed247 
  src/test/python/apache/aurora/executor/test_thermos_task_runner.py 
bb998c0b0ac87ba51ec13e490338b00e7f85e8cf 

Diff: https://reviews.apache.org/r/40201/diff/


Testing
---

Green locally:
```
$ rm -rf dist/ && \
  ./pants test.pytest --no-fast src/test/python/apache/aurora/executor:
```

And no `dist/` created.


Thanks,

John Sirois

Re: Review Request 40201: Cleanup thermos_executor test pexes.

2015-11-11 Thread Aurora ReviewBot 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40201/#review106164
---

Ship it!


Master (0b025a7) is green with this patch.
  ./build-support/jenkins/build.sh

I will refresh this build result if you post a review containing "@ReviewBot 
retry"

- Aurora ReviewBot


On Nov. 11, 2015, 10:11 p.m., John Sirois wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/40201/
> ---
> 
> (Updated Nov. 11, 2015, 10:11 p.m.)
> 
> 
> Review request for Aurora, Kevin Sweeney, Brian Wickman, and Zameer Manji.
> 
> 
> Bugs: AURORA-547
> https://issues.apache.org/jira/browse/AURORA-547
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Previously these were built to the standard `dist/` dir, now they are
> dumped to a tmp dir that's cleaned up.
> 
>  src/test/python/apache/aurora/executor/test_thermos_executor.py| 24 
> +++-
>  src/test/python/apache/aurora/executor/test_thermos_task_runner.py | 15 
> ++-
>  2 files changed, 29 insertions(+), 10 deletions(-)
> 
> 
> Diffs
> -
> 
>   src/test/python/apache/aurora/executor/test_thermos_executor.py 
> d24c61162e11bb4afef2902ebb031478fe3ed247 
>   src/test/python/apache/aurora/executor/test_thermos_task_runner.py 
> bb998c0b0ac87ba51ec13e490338b00e7f85e8cf 
> 
> Diff: https://reviews.apache.org/r/40201/diff/
> 
> 
> Testing
> ---
> 
> Green locally:
> ```
> $ rm -rf dist/ && \
>   ./pants test.pytest --no-fast src/test/python/apache/aurora/executor:
> ```
> 
> And no `dist/` created.
> 
> 
> Thanks,
> 
> John Sirois
> 
>