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

Fair point! This is my first foray into Kotlin so was unaware of the immutable declaration. [ 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] aditya87 commented on pull request #3049: *DO NOT MERGE* Explore building configuration REST service using Kotlin

Yes, definitely would be moving a lot of this stuff out of controller code [ 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] aditya87 commented on pull request #3049: *DO NOT MERGE* Explore building configuration REST service using Kotlin

Sure, that seems like a simple enough thing to fix. [ 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] aditya87 commented on pull request #3049: *DO NOT MERGE* Explore building configuration REST service using Kotlin

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