Uh oh!
There was an error while loading. Please reload this page.
docs(bigquery): remove location parameter from samples - #9500
Conversation
The parameter can sometimes confuse new BigQuery developers. Since location autodetection now works pretty well, the parameter can be removed from code samples for better clarity.
emar-kar
commented
Oct 19, 2019
As I'm working with the refactoring project of the BigQuery samples, your PR might cause conflicts with the already existing one. There is a link to it in the issue. |
@emar-kar Thanks for the notice. I see that the samples have been split into multiple modules. I can make the same set of changes based on the existing PR, and submit my PR to the other PR branch. Then we can merge both set of changes into the main repository without conflicts. |
emar-kar
commented
Oct 19, 2019
It will be great! And you can leave the |
plamut
commented
Oct 19, 2019
Superseded by a pull request opened directly against a related pull request to avoid unnecessary merge conflicts in the samples code. |
Closes#9499.
The parameter can sometimes confuse new BigQuery developers. Since location autodetection now works pretty well, the parameter can be removed from code samples for better clarity.
There didn't seem to be any samples specifically demonstrating how to set the location, thus removed that parameter from all of them.
PR checklist