Allow parsing request and writing responses using java streams #41
Conversation
ansman
force-pushed
the
master
branch
3 times, most recently
from
968dff7 to
0cc75c1
Compare
| @@ -201,6 +201,145 @@ internal class AogRequest internal constructor( | |||
| companion object { | |||
| private val LOG = LoggerFactory.getLogger(AogRequest::class.java.name) | |||
|
|
|||
| private val gson = GsonBuilder() | |||
There was a problem hiding this comment.
Allocating Gson instances semi heavy and should be cached. Ideally one Gson instance for the whole SDK should be used. Since they are immutable it seems like a reasonable thing to do.
There was a problem hiding this comment.
How many times is create called during a particular request? Will it have any performance changes?
There was a problem hiding this comment.
Probably only once, but it will create a new instance for every server call for no good reason. There is no reason not to cache the instance.
There was a problem hiding this comment.
Overall LGTM but I'll probably cherry-pick it and see if I can build a sample with it.
@taycaldwell I'd like your comments as well.
| @@ -201,6 +201,145 @@ internal class AogRequest internal constructor( | |||
| companion object { | |||
| private val LOG = LoggerFactory.getLogger(AogRequest::class.java.name) | |||
|
|
|||
| private val gson = GsonBuilder() | |||
There was a problem hiding this comment.
How many times is create called during a particular request? Will it have any performance changes?
src/test/kotlin/com/google/actions/api/smarthome/SmartHomeResponseTest.kt
Show resolved
Hide resolved
This fixes #39