[GitHub] [geode] Petahhh commented on issue #3049: *DO NOT MERGE* Explore building configuration REST service using Kotlin

2019-01-03 Thread GitHub
Hey @kohlmu-pivotal , 

Could you please rephrase this? I don't fully understand. Specially, could you 
use the phrase "it is **better to** X **instead** of Y"

> I wonder... would it not make more sense to rather treat the REST Controller 
> as an endpoint.. rather put the login inside the controller. Why could we not 
> have access to the ConfigurationService here?



[ Full content available at: https://github.com/apache/geode/pull/3049 ]
This message was relayed via gitbox.apache.org for 
notifications@geode.apache.org


[GitHub] [geode] Petahhh commented on issue #3049: *DO NOT MERGE* Explore building configuration REST service using Kotlin

2019-01-03 Thread GitHub
Hey @kohlmu-pivotal , 

Could you please rephrase this? I don't fully understand

> I wonder... would it not make more sense to rather treat the REST Controller 
> as an endpoint.. rather put the login inside the controller. Why could we not 
> have access to the ConfigurationService here?



[ Full content available at: https://github.com/apache/geode/pull/3049 ]
This message was relayed via gitbox.apache.org for 
notifications@geode.apache.org