Re: Review Request 59210: GEODE-2912: Hot deploy for functions in deployed Jars

2017-05-12 Thread Jinmei Liao

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


Ship it!




Ship It!

- Jinmei Liao


On May 12, 2017, 6:06 p.m., Jared Stewart wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/59210/
> ---
> 
> (Updated May 12, 2017, 6:06 p.m.)
> 
> 
> Review request for geode.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> GEODE-2912: Hot deploy for functions in deployed Jars
> 
> - New versions of a function now deploy over top the old versions without an 
> intermediate undeploy
> 
> 
> Diffs
> -
> 
>   geode-core/src/main/java/org/apache/geode/internal/DeployedJar.java acb7d22 
>   geode-core/src/main/java/org/apache/geode/internal/JarDeployer.java df3f10b 
>   
> geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/DeployCommandRedeployDUnitTest.java
>  6972477 
> 
> 
> Diff: https://reviews.apache.org/r/59210/diff/2/
> 
> 
> Testing
> ---
> 
> Precheckin running
> 
> 
> Thanks,
> 
> Jared Stewart
> 
>



Re: Review Request 59210: GEODE-2912: Hot deploy for functions in deployed Jars

2017-05-12 Thread Ken Howe

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


Ship it!




Ship It!

- Ken Howe


On May 12, 2017, 6:06 p.m., Jared Stewart wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/59210/
> ---
> 
> (Updated May 12, 2017, 6:06 p.m.)
> 
> 
> Review request for geode.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> GEODE-2912: Hot deploy for functions in deployed Jars
> 
> - New versions of a function now deploy over top the old versions without an 
> intermediate undeploy
> 
> 
> Diffs
> -
> 
>   geode-core/src/main/java/org/apache/geode/internal/DeployedJar.java acb7d22 
>   geode-core/src/main/java/org/apache/geode/internal/JarDeployer.java df3f10b 
>   
> geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/DeployCommandRedeployDUnitTest.java
>  6972477 
> 
> 
> Diff: https://reviews.apache.org/r/59210/diff/2/
> 
> 
> Testing
> ---
> 
> Precheckin running
> 
> 
> Thanks,
> 
> Jared Stewart
> 
>



Re: Review Request 59210: GEODE-2912: Hot deploy for functions in deployed Jars

2017-05-12 Thread Jared Stewart

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

(Updated May 12, 2017, 6:06 p.m.)


Review request for geode.


Repository: geode


Description
---

GEODE-2912: Hot deploy for functions in deployed Jars

- New versions of a function now deploy over top the old versions without an 
intermediate undeploy


Diffs (updated)
-

  geode-core/src/main/java/org/apache/geode/internal/DeployedJar.java acb7d22 
  geode-core/src/main/java/org/apache/geode/internal/JarDeployer.java df3f10b 
  
geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/DeployCommandRedeployDUnitTest.java
 6972477 


Diff: https://reviews.apache.org/r/59210/diff/2/

Changes: https://reviews.apache.org/r/59210/diff/1-2/


Testing
---

Precheckin running


Thanks,

Jared Stewart



Re: Review Request 59210: GEODE-2912: Hot deploy for functions in deployed Jars

2017-05-12 Thread Jinmei Liao

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




geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/DeployCommandRedeployDUnitTest.java
Lines 129 (patched)


the rule will automatically disconnect and exit. No need to call this.


- Jinmei Liao


On May 11, 2017, 9:47 p.m., Jared Stewart wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/59210/
> ---
> 
> (Updated May 11, 2017, 9:47 p.m.)
> 
> 
> Review request for geode.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> GEODE-2912: Hot deploy for functions in deployed Jars
> 
> - New versions of a function now deploy over top the old versions without an 
> intermediate undeploy
> 
> 
> Diffs
> -
> 
>   geode-core/src/main/java/org/apache/geode/internal/DeployedJar.java 
> acb7d227a4f67c749cbc11ee2fdae8651d3bc5d6 
>   geode-core/src/main/java/org/apache/geode/internal/JarDeployer.java 
> df3f10b8cba9bdca8429bdcf8567b654d36ea475 
>   
> geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/DeployCommandRedeployDUnitTest.java
>  697247701f883a5532ac0118ff116ac6562776d4 
> 
> 
> Diff: https://reviews.apache.org/r/59210/diff/1/
> 
> 
> Testing
> ---
> 
> Precheckin running
> 
> 
> Thanks,
> 
> Jared Stewart
> 
>



Re: Review Request 59210: GEODE-2912: Hot deploy for functions in deployed Jars

2017-05-12 Thread Ken Howe

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




geode-core/src/main/java/org/apache/geode/internal/DeployedJar.java
Lines 219-220 (patched)


minor: extraneous blank lines



geode-core/src/main/java/org/apache/geode/internal/DeployedJar.java
Lines 233 (patched)


Can't the newVesrion == null test be pulled outside the for loop?


if (newVersion != null) {
  for ...
}




geode-core/src/main/java/org/apache/geode/internal/DeployedJar.java
Lines 409 (patched)


CollectionUtils.isEmpty(this.registeredFunctions) will handle the null 
case, and it's more concise.



geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/DeployCommandRedeployDUnitTest.java
Lines 129 (patched)


Put this in an @After block


- Ken Howe


On May 11, 2017, 9:47 p.m., Jared Stewart wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/59210/
> ---
> 
> (Updated May 11, 2017, 9:47 p.m.)
> 
> 
> Review request for geode.
> 
> 
> Repository: geode
> 
> 
> Description
> ---
> 
> GEODE-2912: Hot deploy for functions in deployed Jars
> 
> - New versions of a function now deploy over top the old versions without an 
> intermediate undeploy
> 
> 
> Diffs
> -
> 
>   geode-core/src/main/java/org/apache/geode/internal/DeployedJar.java 
> acb7d227a4f67c749cbc11ee2fdae8651d3bc5d6 
>   geode-core/src/main/java/org/apache/geode/internal/JarDeployer.java 
> df3f10b8cba9bdca8429bdcf8567b654d36ea475 
>   
> geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/DeployCommandRedeployDUnitTest.java
>  697247701f883a5532ac0118ff116ac6562776d4 
> 
> 
> Diff: https://reviews.apache.org/r/59210/diff/1/
> 
> 
> Testing
> ---
> 
> Precheckin running
> 
> 
> Thanks,
> 
> Jared Stewart
> 
>



Review Request 59210: GEODE-2912: Hot deploy for functions in deployed Jars

2017-05-11 Thread Jared Stewart

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

Review request for geode.


Repository: geode


Description
---

GEODE-2912: Hot deploy for functions in deployed Jars

- New versions of a function now deploy over top the old versions without an 
intermediate undeploy


Diffs
-

  geode-core/src/main/java/org/apache/geode/internal/DeployedJar.java 
acb7d227a4f67c749cbc11ee2fdae8651d3bc5d6 
  geode-core/src/main/java/org/apache/geode/internal/JarDeployer.java 
df3f10b8cba9bdca8429bdcf8567b654d36ea475 
  
geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/DeployCommandRedeployDUnitTest.java
 697247701f883a5532ac0118ff116ac6562776d4 


Diff: https://reviews.apache.org/r/59210/diff/1/


Testing
---

Precheckin running


Thanks,

Jared Stewart