api samples: Update samples to 1.16.0-rc

api samples: Update samples to 1.16.0-rc

The updated jars for the Android samples have not been included in this codereview, as they are too large.

The following samples have been tested:
calendar-cmdline-sample
plus-appengine-sample
calendar-android-sample

[peleyal, 2013-07-19 19:45:51]

https://codereview.appspot.com/11587043/diff/1/adsensehost-cmdline-sample/src...
File
adsensehost-cmdline-sample/src/main/java/com/google/api/services/samples/adsensehost/cmdline/AdSenseHostSample.java
(right):

https://codereview.appspot.com/11587043/diff/1/adsensehost-cmdline-sample/src...
adsensehost-cmdline-sample/src/main/java/com/google/api/services/samples/adsensehost/cmdline/AdSenseHostSample.java:108:
new FileDataStoreFactory(DATA_STORE_DIR)).build();
I think we should store that DataStoreFactory in a variable, in that case we
will add a comment like the one in DataStoreFactory (Users should keep a single
globally shared instance of the data store factory.).

https://codereview.appspot.com/11587043/diff/1/analytics-cmdline-sample/src/m...
File
analytics-cmdline-sample/src/main/java/com/google/api/services/samples/analytics/cmdline/CoreReportingApiReferenceSample.java
(right):

https://codereview.appspot.com/11587043/diff/1/analytics-cmdline-sample/src/m...
analytics-cmdline-sample/src/main/java/com/google/api/services/samples/analytics/cmdline/CoreReportingApiReferenceSample.java:44:
*  To run this, you must supply your Google Analytics TABLE ID. Read the Core
Reporting API
remove extra sapce

https://codereview.appspot.com/11587043/diff/1/analytics-cmdline-sample/src/m...
File
analytics-cmdline-sample/src/main/java/com/google/api/services/samples/analytics/cmdline/ManagementApiReferenceSample.java
(right):

https://codereview.appspot.com/11587043/diff/1/analytics-cmdline-sample/src/m...
analytics-cmdline-sample/src/main/java/com/google/api/services/samples/analytics/cmdline/ManagementApiReferenceSample.java:54:
*  Note: This demo does not store OAuth 2.0 refresh tokens. Each time the sample
is run, the user
Extra space, please remove. You can also wrap it in <p>

https://codereview.appspot.com/11587043/diff/1/analytics-cmdline-sample/src/m...
File
analytics-cmdline-sample/src/main/java/com/google/api/services/samples/analytics/cmdline/McfReportingApiReferenceSample.java
(right):

https://codereview.appspot.com/11587043/diff/1/analytics-cmdline-sample/src/m...
analytics-cmdline-sample/src/main/java/com/google/api/services/samples/analytics/cmdline/McfReportingApiReferenceSample.java:63:
new java.io.File(System.getProperty("user.home"),
".store/analytics_mcf_reporting_sample");
Share store for all analytics samples.
e.g. analytics_mcf_reporting_sample -> analytics_sample

https://codereview.appspot.com/11587043/diff/1/bigquery-appengine-sample/src/...
File
bigquery-appengine-sample/src/main/java/com/google/api/client/sample/bigquery/appengine/dashboard/ServiceUtils.java
(right):

https://codereview.appspot.com/11587043/diff/1/bigquery-appengine-sample/src/...
bigquery-appengine-sample/src/main/java/com/google/api/client/sample/bigquery/appengine/dashboard/ServiceUtils.java:51:
static final AppEngineDataStoreFactory DATA_STORE_FACTORY = new
AppEngineDataStoreFactory();
You see... here you actually save that variable. We should do so in all our cmd
samples as well

https://codereview.appspot.com/11587043/diff/1/calendar-appengine-sample/pom.xml
File calendar-appengine-sample/pom.xml (right):

https://codereview.appspot.com/11587043/diff/1/calendar-appengine-sample/pom....
calendar-appengine-sample/pom.xml:289: <groupId>com.google.api-client</groupId>
Just curious - why do we need to add that?

https://codereview.appspot.com/11587043/diff/1/calendar-cmdline-sample/src/ma...
File
calendar-cmdline-sample/src/main/java/com/google/api/services/samples/calendar/cmdline/CalendarSample.java
(right):

https://codereview.appspot.com/11587043/diff/1/calendar-cmdline-sample/src/ma...
calendar-cmdline-sample/src/main/java/com/google/api/services/samples/calendar/cmdline/CalendarSample.java:138:
@Override
I know that eclipse sucks! but please remove the extra space in 138, 144, 210,
215

https://codereview.appspot.com/11587043/diff/1/plus-appengine-sample/src/main...
File
plus-appengine-sample/src/main/java/com/google/api/services/samples/plus/Utils.java
(right):

https://codereview.appspot.com/11587043/diff/1/plus-appengine-sample/src/main...
plus-appengine-sample/src/main/java/com/google/api/services/samples/plus/Utils.java:57:
new AppEngineDataStoreFactory()).setAccessType("offline").build();
store in a variable here as well

https://codereview.appspot.com/11587043/diff/1/storage-cmdline-sample/src/mai...
File
storage-cmdline-sample/src/main/java/com/google/api/services/samples/storage/cmdline/StorageSample.java
(right):

https://codereview.appspot.com/11587043/diff/1/storage-cmdline-sample/src/mai...
storage-cmdline-sample/src/main/java/com/google/api/services/samples/storage/cmdline/StorageSample.java:255:
// .setDefaultObjectAcl(ImmutableList.of(
What do you think? Should we remove that commented code?

https://codereview.appspot.com/11587043/diff/1/storage-cmdline-sample/src/mai...
storage-cmdline-sample/src/main/java/com/google/api/services/samples/storage/cmdline/StorageSample.java:496:
new HttpHeaders().setRange(String.format("bytes=%d-%d", 2 * 1000 * 1000, 3 *
1000 * 1000 - 1 /*
make it one line, you can use "//"

https://codereview.appspot.com/11587043/diff/1/taskqueue-cmdline-sample/src/m...
File
taskqueue-cmdline-sample/src/main/java/com/google/api/services/samples/taskqueue/cmdline/TaskQueueSample.java
(right):

https://codereview.appspot.com/11587043/diff/1/taskqueue-cmdline-sample/src/m...
taskqueue-cmdline-sample/src/main/java/com/google/api/services/samples/taskqueue/cmdline/TaskQueueSample.java:103:
@Override
Eclipse sucks (2)

https://codereview.appspot.com/11587043/diff/1/urlshortener-robots-appengine-...
File
urlshortener-robots-appengine-sample/src/main/java/com/google/api/client/sample/urlshortener/appengine/robots/UrlShortenerSample.java
(right):

https://codereview.appspot.com/11587043/diff/1/urlshortener-robots-appengine-...
urlshortener-robots-appengine-sample/src/main/java/com/google/api/client/sample/urlshortener/appengine/robots/UrlShortenerSample.java:20:
import com.google.api.client.json.jackson2.JacksonFactory;
Why?

[ngmiceli, 2013-07-19 21:57:33]

Didn't finish the DataStoreFactory refactoring by EOD. Will continue on Monday.

https://codereview.appspot.com/11587043/diff/1/analytics-cmdline-sample/src/m...
File
analytics-cmdline-sample/src/main/java/com/google/api/services/samples/analytics/cmdline/CoreReportingApiReferenceSample.java
(right):

https://codereview.appspot.com/11587043/diff/1/analytics-cmdline-sample/src/m...
analytics-cmdline-sample/src/main/java/com/google/api/services/samples/analytics/cmdline/CoreReportingApiReferenceSample.java:44:
*  To run this, you must supply your Google Analytics TABLE ID. Read the Core
Reporting API
On 2013/07/19 19:45:51, peleyal wrote:
> remove extra sapce

Done.

https://codereview.appspot.com/11587043/diff/1/analytics-cmdline-sample/src/m...
File
analytics-cmdline-sample/src/main/java/com/google/api/services/samples/analytics/cmdline/ManagementApiReferenceSample.java
(right):

https://codereview.appspot.com/11587043/diff/1/analytics-cmdline-sample/src/m...
analytics-cmdline-sample/src/main/java/com/google/api/services/samples/analytics/cmdline/ManagementApiReferenceSample.java:54:
*  Note: This demo does not store OAuth 2.0 refresh tokens. Each time the sample
is run, the user
On 2013/07/19 19:45:51, peleyal wrote:
> Extra space, please remove. You can also wrap it in <p>

Done.

https://codereview.appspot.com/11587043/diff/1/analytics-cmdline-sample/src/m...
File
analytics-cmdline-sample/src/main/java/com/google/api/services/samples/analytics/cmdline/McfReportingApiReferenceSample.java
(right):

https://codereview.appspot.com/11587043/diff/1/analytics-cmdline-sample/src/m...
analytics-cmdline-sample/src/main/java/com/google/api/services/samples/analytics/cmdline/McfReportingApiReferenceSample.java:63:
new java.io.File(System.getProperty("user.home"),
".store/analytics_mcf_reporting_sample");
On 2013/07/19 19:45:51, peleyal wrote:
> Share store for all analytics samples.
> e.g. analytics_mcf_reporting_sample -> analytics_sample

Done.

https://codereview.appspot.com/11587043/diff/1/bigquery-appengine-sample/src/...
File
bigquery-appengine-sample/src/main/java/com/google/api/client/sample/bigquery/appengine/dashboard/ServiceUtils.java
(right):

https://codereview.appspot.com/11587043/diff/1/bigquery-appengine-sample/src/...
bigquery-appengine-sample/src/main/java/com/google/api/client/sample/bigquery/appengine/dashboard/ServiceUtils.java:51:
static final AppEngineDataStoreFactory DATA_STORE_FACTORY = new
AppEngineDataStoreFactory();
On 2013/07/19 19:45:51, peleyal wrote:
> You see... here you actually save that variable. We should do so in all our
cmd
> samples as well

Acknowledged.

https://codereview.appspot.com/11587043/diff/1/calendar-appengine-sample/pom.xml
File calendar-appengine-sample/pom.xml (right):

https://codereview.appspot.com/11587043/diff/1/calendar-appengine-sample/pom....
calendar-appengine-sample/pom.xml:289: <groupId>com.google.api-client</groupId>
On 2013/07/19 19:45:51, peleyal wrote:
> Just curious - why do we need to add that?

We're upgrading the import from jackson to jackson2, and that's not included as
a transitive dependency of anything else in this list. Check out the changes to
the source code for this sample.

https://codereview.appspot.com/11587043/diff/1/calendar-cmdline-sample/src/ma...
File
calendar-cmdline-sample/src/main/java/com/google/api/services/samples/calendar/cmdline/CalendarSample.java
(right):

https://codereview.appspot.com/11587043/diff/1/calendar-cmdline-sample/src/ma...
calendar-cmdline-sample/src/main/java/com/google/api/services/samples/calendar/cmdline/CalendarSample.java:138:
@Override
On 2013/07/19 19:45:51, peleyal wrote:
> I know that eclipse sucks! but please remove the extra space in 138, 144, 210,
> 215

It borders on a no-comment issue because every time we update the samples, if we
use the auto-formatter this is going to happen. From the style guide: "The costs
of attempting to banish [trailing whitespace] exceed the benefits; the most
productive course of action is to just tolerate it"
Nonetheless, it is a sample and so readability is a high-priority, so done. :)

https://codereview.appspot.com/11587043/diff/1/storage-cmdline-sample/src/mai...
File
storage-cmdline-sample/src/main/java/com/google/api/services/samples/storage/cmdline/StorageSample.java
(right):

https://codereview.appspot.com/11587043/diff/1/storage-cmdline-sample/src/mai...
storage-cmdline-sample/src/main/java/com/google/api/services/samples/storage/cmdline/StorageSample.java:255:
// .setDefaultObjectAcl(ImmutableList.of(
On 2013/07/19 19:45:51, peleyal wrote:
> What do you think? Should we remove that commented code?

The problem is Nathan didn't explain why it's here. Perhaps a user may find it
really useful to uncomment it and run it and see the difference. I'm not sure. 
My instinct is just to leave it. I'm frustrated enough that I have to touch the
code because of the auto-formatter. My goal is just to change whatever is needed
to have it work with the new version and nothing else.

https://codereview.appspot.com/11587043/diff/1/storage-cmdline-sample/src/mai...
storage-cmdline-sample/src/main/java/com/google/api/services/samples/storage/cmdline/StorageSample.java:496:
new HttpHeaders().setRange(String.format("bytes=%d-%d", 2 * 1000 * 1000, 3 *
1000 * 1000 - 1 /*
On 2013/07/19 19:45:51, peleyal wrote:
> make it one line, you can use "//"

Haha what a weird one

https://codereview.appspot.com/11587043/diff/1/taskqueue-cmdline-sample/src/m...
File
taskqueue-cmdline-sample/src/main/java/com/google/api/services/samples/taskqueue/cmdline/TaskQueueSample.java
(right):

https://codereview.appspot.com/11587043/diff/1/taskqueue-cmdline-sample/src/m...
taskqueue-cmdline-sample/src/main/java/com/google/api/services/samples/taskqueue/cmdline/TaskQueueSample.java:103:
@Override
On 2013/07/19 19:45:51, peleyal wrote:
> Eclipse sucks (2)

Now you're just hurting his feelings. Done.

https://codereview.appspot.com/11587043/diff/1/urlshortener-robots-appengine-...
File
urlshortener-robots-appengine-sample/src/main/java/com/google/api/client/sample/urlshortener/appengine/robots/UrlShortenerSample.java
(right):

https://codereview.appspot.com/11587043/diff/1/urlshortener-robots-appengine-...
urlshortener-robots-appengine-sample/src/main/java/com/google/api/client/sample/urlshortener/appengine/robots/UrlShortenerSample.java:20:
import com.google.api.client.json.jackson2.JacksonFactory;
On 2013/07/19 19:45:51, peleyal wrote:
> Why?

Jackson2.0 is preferred over Jackson1.X

[yanivi, 2013-07-22 12:52:25]

https://codereview.appspot.com/11587043/diff/12006/bigquery-appengine-sample/...
File bigquery-appengine-sample/pom.xml (right):

https://codereview.appspot.com/11587043/diff/12006/bigquery-appengine-sample/...
bigquery-appengine-sample/pom.xml:57: <artifactId>maven-gae-plugin</artifactId>
we should take the time to update all of the appengine samples to
appengine-maven-plugin (version 1.8.2) and update instructions on how to deploy
to appspot

see plus-appengine-sample

[ngmiceli, 2013-07-22 15:58:30]

On 2013/07/22 12:52:25, yanivi wrote:
>
https://codereview.appspot.com/11587043/diff/12006/bigquery-appengine-sample/...
> File bigquery-appengine-sample/pom.xml (right):
> 
>
https://codereview.appspot.com/11587043/diff/12006/bigquery-appengine-sample/...
> bigquery-appengine-sample/pom.xml:57:
<artifactId>maven-gae-plugin</artifactId>
> we should take the time to update all of the appengine samples to
> appengine-maven-plugin (version 1.8.2) and update instructions on how to
deploy
> to appspot
> 
> see plus-appengine-sample

Done.

[ngmiceli, 2013-07-22 15:59:57]

PTAL

https://codereview.appspot.com/11587043/diff/1/adsensehost-cmdline-sample/src...
File
adsensehost-cmdline-sample/src/main/java/com/google/api/services/samples/adsensehost/cmdline/AdSenseHostSample.java
(right):

https://codereview.appspot.com/11587043/diff/1/adsensehost-cmdline-sample/src...
adsensehost-cmdline-sample/src/main/java/com/google/api/services/samples/adsensehost/cmdline/AdSenseHostSample.java:108:
new FileDataStoreFactory(DATA_STORE_DIR)).build();
On 2013/07/19 19:45:51, peleyal wrote:
> I think we should store that DataStoreFactory in a variable, in that case we
> will add a comment like the one in DataStoreFactory (Users should keep a
single
> globally shared instance of the data store factory.).

Done.

https://codereview.appspot.com/11587043/diff/1/plus-appengine-sample/src/main...
File
plus-appengine-sample/src/main/java/com/google/api/services/samples/plus/Utils.java
(right):

https://codereview.appspot.com/11587043/diff/1/plus-appengine-sample/src/main...
plus-appengine-sample/src/main/java/com/google/api/services/samples/plus/Utils.java:57:
new AppEngineDataStoreFactory()).setAccessType("offline").build();
On 2013/07/19 19:45:51, peleyal wrote:
> store in a variable here as well

Done.

[peleyal, 2013-07-22 18:56:55]

Like Yaniv likes to say: almost there :)
Let's finish with that today!

https://codereview.appspot.com/11587043/diff/1/bigquery-appengine-sample/pom.xml
File bigquery-appengine-sample/pom.xml (right):

https://codereview.appspot.com/11587043/diff/1/bigquery-appengine-sample/pom....
bigquery-appengine-sample/pom.xml:17: <pluginRepository>
why did you delete it?

https://codereview.appspot.com/11587043/diff/1/calendar-appengine-sample/pom.xml
File calendar-appengine-sample/pom.xml (right):

https://codereview.appspot.com/11587043/diff/1/calendar-appengine-sample/pom....
calendar-appengine-sample/pom.xml:95: <groupId>net.kindleit</groupId>
Why did you remove it? Did the AppEngine team updated their plugin?

https://codereview.appspot.com/11587043/diff/1/taskqueue-cmdline-sample/src/m...
File
taskqueue-cmdline-sample/src/main/java/com/google/api/services/samples/taskqueue/cmdline/TaskQueueSample.java
(right):

https://codereview.appspot.com/11587043/diff/1/taskqueue-cmdline-sample/src/m...
taskqueue-cmdline-sample/src/main/java/com/google/api/services/samples/taskqueue/cmdline/TaskQueueSample.java:103:
@Override
Sorry Eclipse, but let's admit it - you suck!
On 2013/07/19 21:57:33, ngmiceli wrote:
> On 2013/07/19 19:45:51, peleyal wrote:
> > Eclipse sucks (2)
> 
> Now you're just hurting his feelings. Done.

https://codereview.appspot.com/11587043/diff/82001/plus-appengine-sample/.set...
File plus-appengine-sample/.settings/com.google.appengine.eclipse.core.prefs
(right):

https://codereview.appspot.com/11587043/diff/82001/plus-appengine-sample/.set...
plus-appengine-sample/.settings/com.google.appengine.eclipse.core.prefs:2:
filesCopiedToWebInfLib=appengine-api-1.0-sdk-1.8.2.jar|appengine-api-labs.jar|appengine-endpoints.jar|appengine-jsr107cache-1.8.2.jar|datanucleus-appengine-1.0.10.final.jar|datanucleus-core-1.1.5.jar|datanucleus-jpa-1.1.5.jar|geronimo-jpa_3.0_spec-1.1.1.jar|geronimo-jta_1.1_spec-1.1.1.jar|jdo2-api-2.3-eb.jar|jsr107cache-1.1.jar
Do you need to update the instructions.html file of this sample as well?

https://codereview.appspot.com/11587043/diff/82001/taskqueue-cmdline-sample/s...
File
taskqueue-cmdline-sample/src/main/java/com/google/api/services/samples/taskqueue/cmdline/TaskQueueSample.java
(right):

https://codereview.appspot.com/11587043/diff/82001/taskqueue-cmdline-sample/s...
taskqueue-cmdline-sample/src/main/java/com/google/api/services/samples/taskqueue/cmdline/TaskQueueSample.java:102:
* <p>
please close using </ul> + ENTER

https://codereview.appspot.com/11587043/diff/82001/urlshortener-robots-appeng...
File
urlshortener-robots-appengine-sample/.settings/com.google.appengine.eclipse.core.prefs
(right):

https://codereview.appspot.com/11587043/diff/82001/urlshortener-robots-appeng...
urlshortener-robots-appengine-sample/.settings/com.google.appengine.eclipse.core.prefs:2:
filesCopiedToWebInfLib=appengine-api-1.0-sdk-1.8.2.jar|appengine-api-labs.jar|appengine-endpoints.jar|appengine-jsr107cache-1.8.2.jar|datanucleus-appengine-1.0.10.final.jar|datanucleus-core-1.1.5.jar|datanucleus-jpa-1.1.5.jar|geronimo-jpa_3.0_spec-1.1.1.jar|geronimo-jta_1.1_spec-1.1.1.jar|jdo2-api-2.3-eb.jar|jsr107cache-1.1.jar
Do you need to update the instructions.html file of this sample as well?

[ngmiceli, 2013-07-22 19:31:05]

https://codereview.appspot.com/11587043/diff/82001/plus-appengine-sample/.set...
File plus-appengine-sample/.settings/com.google.appengine.eclipse.core.prefs
(right):

https://codereview.appspot.com/11587043/diff/82001/plus-appengine-sample/.set...
plus-appengine-sample/.settings/com.google.appengine.eclipse.core.prefs:2:
filesCopiedToWebInfLib=appengine-api-1.0-sdk-1.8.2.jar|appengine-api-labs.jar|appengine-endpoints.jar|appengine-jsr107cache-1.8.2.jar|datanucleus-appengine-1.0.10.final.jar|datanucleus-core-1.1.5.jar|datanucleus-jpa-1.1.5.jar|geronimo-jpa_3.0_spec-1.1.1.jar|geronimo-jta_1.1_spec-1.1.1.jar|jdo2-api-2.3-eb.jar|jsr107cache-1.1.jar
On 2013/07/22 18:56:56, peleyal wrote:
> Do you need to update the instructions.html file of this sample as well?

Nope. I wrote this sample using the correct plugin from the start.

See:
https://code.google.com/p/google-api-java-client/source/browse/plus-appengine...

https://codereview.appspot.com/11587043/diff/82001/taskqueue-cmdline-sample/s...
File
taskqueue-cmdline-sample/src/main/java/com/google/api/services/samples/taskqueue/cmdline/TaskQueueSample.java
(right):

https://codereview.appspot.com/11587043/diff/82001/taskqueue-cmdline-sample/s...
taskqueue-cmdline-sample/src/main/java/com/google/api/services/samples/taskqueue/cmdline/TaskQueueSample.java:102:
* <p>
On 2013/07/22 18:56:56, peleyal wrote:
> please close using </ul> + ENTER
Nice catch. Done.

https://codereview.appspot.com/11587043/diff/82001/urlshortener-robots-appeng...
File
urlshortener-robots-appengine-sample/.settings/com.google.appengine.eclipse.core.prefs
(right):

https://codereview.appspot.com/11587043/diff/82001/urlshortener-robots-appeng...
urlshortener-robots-appengine-sample/.settings/com.google.appengine.eclipse.core.prefs:2:
filesCopiedToWebInfLib=appengine-api-1.0-sdk-1.8.2.jar|appengine-api-labs.jar|appengine-endpoints.jar|appengine-jsr107cache-1.8.2.jar|datanucleus-appengine-1.0.10.final.jar|datanucleus-core-1.1.5.jar|datanucleus-jpa-1.1.5.jar|geronimo-jpa_3.0_spec-1.1.1.jar|geronimo-jta_1.1_spec-1.1.1.jar|jdo2-api-2.3-eb.jar|jsr107cache-1.1.jar
On 2013/07/22 18:56:56, peleyal wrote:
> Do you need to update the instructions.html file of this sample as well?

Done.

[peleyal, 2013-07-22 19:39:59]

LGTM, just please correct my two silly comments

https://codereview.appspot.com/11587043/diff/82001/bigquery-appengine-sample/...
File bigquery-appengine-sample/pom.xml (right):

https://codereview.appspot.com/11587043/diff/82001/bigquery-appengine-sample/...
bigquery-appengine-sample/pom.xml:46: 
please remove

https://codereview.appspot.com/11587043/diff/82001/taskqueue-cmdline-sample/s...
File
taskqueue-cmdline-sample/src/main/java/com/google/api/services/samples/taskqueue/cmdline/TaskQueueSample.java
(right):

https://codereview.appspot.com/11587043/diff/82001/taskqueue-cmdline-sample/s...
taskqueue-cmdline-sample/src/main/java/com/google/api/services/samples/taskqueue/cmdline/TaskQueueSample.java:102:
* <p>
[Optional] make 103 empty line :)
On 2013/07/22 19:31:06, ngmiceli wrote:
> On 2013/07/22 18:56:56, peleyal wrote:
> > please close using </ul> + ENTER
> Nice catch. Done.

[ngmiceli, 2013-07-22 19:45:54]

https://codereview.appspot.com/11587043/diff/1/bigquery-appengine-sample/pom.xml
File bigquery-appengine-sample/pom.xml (right):

https://codereview.appspot.com/11587043/diff/1/bigquery-appengine-sample/pom....
bigquery-appengine-sample/pom.xml:17: <pluginRepository>
On 2013/07/22 18:56:56, peleyal wrote:
> why did you delete it?

This is the old plugin we used for the GAE SDK. It wasn't an official Google App
Engine plugin. It's been replaced by the official one which has all of the same
functionality (but slightly different commands, thus the update to the
instructions.html files).

https://codereview.appspot.com/11587043/diff/12006/bigquery-appengine-sample/...
File bigquery-appengine-sample/pom.xml (right):

https://codereview.appspot.com/11587043/diff/12006/bigquery-appengine-sample/...
bigquery-appengine-sample/pom.xml:57: <artifactId>maven-gae-plugin</artifactId>
On 2013/07/22 12:52:26, yanivi wrote:
> we should take the time to update all of the appengine samples to
> appengine-maven-plugin (version 1.8.2) and update instructions on how to
deploy
> to appspot
> 
> see plus-appengine-sample

Done.

https://codereview.appspot.com/11587043/diff/82001/bigquery-appengine-sample/...
File bigquery-appengine-sample/pom.xml (right):

https://codereview.appspot.com/11587043/diff/82001/bigquery-appengine-sample/...
bigquery-appengine-sample/pom.xml:46: 
On 2013/07/22 19:39:59, peleyal wrote:
> please remove

Done, but I'm not uploading again for this.

https://codereview.appspot.com/11587043/diff/82001/taskqueue-cmdline-sample/s...
File
taskqueue-cmdline-sample/src/main/java/com/google/api/services/samples/taskqueue/cmdline/TaskQueueSample.java
(right):

https://codereview.appspot.com/11587043/diff/82001/taskqueue-cmdline-sample/s...
taskqueue-cmdline-sample/src/main/java/com/google/api/services/samples/taskqueue/cmdline/TaskQueueSample.java:102:
* <p>
On 2013/07/22 19:39:59, peleyal wrote:
> [Optional] make 103 empty line :)
> On 2013/07/22 19:31:06, ngmiceli wrote:
> > On 2013/07/22 18:56:56, peleyal wrote:
> > > please close using </ul> + ENTER
> > Nice catch. Done.
> 

Done

[yanivi, 2013-07-22 20:49:46]

On 2013/07/22 15:58:30, ngmiceli wrote:
> On 2013/07/22 12:52:25, yanivi wrote:
> >
>
https://codereview.appspot.com/11587043/diff/12006/bigquery-appengine-sample/...
> > File bigquery-appengine-sample/pom.xml (right):
> > 
> >
>
https://codereview.appspot.com/11587043/diff/12006/bigquery-appengine-sample/...
> > bigquery-appengine-sample/pom.xml:57:
> <artifactId>maven-gae-plugin</artifactId>
> > we should take the time to update all of the appengine samples to
> > appengine-maven-plugin (version 1.8.2) and update instructions on how to
> deploy
> > to appspot
> > 
> > see plus-appengine-sample
> 
> Done.

Awesome, thanks Nick!