-
Notifications
You must be signed in to change notification settings - Fork 174
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
reduce search results size #641
Comments
Could you share a recent Also, are you using the new Thanks |
Hi Josh, yes, I use the new parameter (thanks again) from the very beginning. What I did to find some examples is to sum up SearchGoogleAdsStreamResponse.getSerializedSize() for each request. I didn't make it to 280MB, but here are three examples: requestId=qTODgOHB8he5i0kXYfU5Rw, size=226.617.663 bytes, query=
requestId=R7ZPX_DmbxpDlpNlLInbXg, size=175.483.361 bytes, query=
requestId=lamT2O1-dPfI4o0P67VyeQ, size=195.195.731 bytes, query=
The problem is not that I get this big results sizes for a single account, the problem is, that I do the same request for hundred(s) of accounts - so even if the result is "only" 100MB large, for >100 accounts you easily run into Gigabytes. Best regards, Christian |
Describe the problem you are trying to solve
I migrated from old Adwords API to new Ads API. I recognized that loading some of my reports take longer than before. First I thought it was an server issues, but then I found out that the responses are much larger. An example: when doing the request
on a specific account, the response stream contains 280MB of data. When I did the same request with the old Adwords Reporting API (using GZIPPED_CSV as download format) the download size was only 200KB.
Describe the solution you would like
It would be great if the response size of the Ads API could be similar to the report size of the Adwords API and not differ by a factor of 1400. The old API returned a compressed csv file, the response of the new API seems to be uncompressed. It would be great if the Ads API would communicate to the server using compressed messages.
Describe alternatives you've considered
There is not much I can do on my side, but I hope the library could
io.grpc.stub.AbstractStub.withCompression("gzip")
Additional context
I copied the most of this ticket's content from #438 that I wrote one year ago. The old issue was closed, but only one part of the request was implemented (thanks for that again). @nwbirnie wrote that I should check back in a few months.
The text was updated successfully, but these errors were encountered: