Fix flaky DLP inspect tests#2359
Conversation
lesv
left a comment
There was a problem hiding this comment.
It looks Ok, and compiles, but I'm confused as I don't see the a reference to handleMessage (only the removal of it), nor do I see the code for messageHandler, though it's being set where handleMessage used to be.
I wonder if there shouldn't be some additional comments, but if you think about my comments and are still ok, feel free to merge.
|
The method definition for |
* refactored and increased time limit for inspect samples
Increased the timeout period for the sample from 10 to 15 min, to resolve #2302 , #2303 , and #2317. Also refactored the Pub/Sub code in the samples to be more consistent with other samples.