Convert aws sdk 2.2 sqs tracing tests from groovy to java#11240
Conversation
| return map; | ||
| } | ||
|
|
||
| ReceiveMessageRequest receiveMessageRequest = |
There was a problem hiding this comment.
It had better to define the those fields in a class together.
There was a problem hiding this comment.
could you clarify what you mean for this one? I moved the queueUrl field closer to these methods that use it, but I'm not sure if that's what you are suggesting. Are you saying I should move these request fields into a new class?
There was a problem hiding this comment.
could you clarify what you mean for this one? I moved the
queueUrlfield closer to these methods that use it, but I'm not sure if that's what you are suggesting. Are you saying I should move these request fields into a new class?
What I actually mean is that all the fields in a class are defined in a piece of code, and then it is followed by some method definitions. I personally think this will make a class look clearer, for example, it will be easier to check whether some fields are used or whether the writing complies with the specification:)
There was a problem hiding this comment.
that makes sense, thank you for the explanation
Related to #7195