You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geode.apache.org by Jason Huynh <hu...@gmail.com> on 2016/06/10 15:12:21 UTC

Review Request 48556: Fixing spark connector build issue due to ConfigurationProperties name change

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

Review request for geode, Udo Kohlmeyer and Dan Smith.


Repository: geode


Description
-------

Modified JavaApiIntegrationTest as it was pointing to DistributedSystemConfigProperties to the renamed ConfigurationProperties


Diffs
-----

  geode-spark-connector/geode-spark-connector/src/it/java/ittest/io/pivotal/geode/spark/connector/JavaApiIntegrationTest.java 1bae89b 

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


Testing
-------


Thanks,

Jason Huynh


Re: Review Request 48556: Fixing spark connector build issue due to ConfigurationProperties name change

Posted by Udo Kohlmeyer <uk...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48556/#review137554
-----------------------------------------------------------


Ship it!




Ship It!

- Udo Kohlmeyer


On June 10, 2016, 3:12 p.m., Jason Huynh wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48556/
> -----------------------------------------------------------
> 
> (Updated June 10, 2016, 3:12 p.m.)
> 
> 
> Review request for geode, Udo Kohlmeyer and Dan Smith.
> 
> 
> Repository: geode
> 
> 
> Description
> -------
> 
> Modified JavaApiIntegrationTest as it was pointing to DistributedSystemConfigProperties to the renamed ConfigurationProperties
> 
> 
> Diffs
> -----
> 
>   geode-spark-connector/geode-spark-connector/src/it/java/ittest/io/pivotal/geode/spark/connector/JavaApiIntegrationTest.java 1bae89b 
> 
> Diff: https://reviews.apache.org/r/48556/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jason Huynh
> 
>


Re: Review Request 48556: Fixing spark connector build issue due to ConfigurationProperties name change

Posted by Dan Smith <ds...@pivotal.io>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48556/#review137021
-----------------------------------------------------------


Ship it!




Ship It!

- Dan Smith


On June 10, 2016, 3:12 p.m., Jason Huynh wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48556/
> -----------------------------------------------------------
> 
> (Updated June 10, 2016, 3:12 p.m.)
> 
> 
> Review request for geode, Udo Kohlmeyer and Dan Smith.
> 
> 
> Repository: geode
> 
> 
> Description
> -------
> 
> Modified JavaApiIntegrationTest as it was pointing to DistributedSystemConfigProperties to the renamed ConfigurationProperties
> 
> 
> Diffs
> -----
> 
>   geode-spark-connector/geode-spark-connector/src/it/java/ittest/io/pivotal/geode/spark/connector/JavaApiIntegrationTest.java 1bae89b 
> 
> Diff: https://reviews.apache.org/r/48556/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jason Huynh
> 
>