Reducing log chatter in picasa sync

diff --git a/src/com/cooliris/picasa/PicasaApi.java b/src/com/cooliris/picasa/PicasaApi.java
index 15a9f20..01ab1ed 100644 (file)
--- a/src/com/cooliris/picasa/PicasaApi.java
+++ b/src/com/cooliris/picasa/PicasaApi.java
@@ -131,8 +131,6 @@ public final class PicasaApi {
         builder.append(Uri.encode(mAuth.user));
         builder.append(BASE_QUERY_STRING);
         builder.append("&kind=album");
-        Log.i(TAG, "getAlbums uri " + builder.toString());
-
         try {
             // Send the request.
             synchronized (mOperation) {
@@ -160,6 +158,7 @@ public final class PicasaApi {
                             ++syncResult.stats.numAuthExceptions;
                         }
                     default:
+                        Log.i(TAG, "getAlbums uri " + builder.toString());
                         Log.e(TAG, "getAlbums: unexpected status code " + operation.outStatus + " data: "
                                 + operation.outBody.toString());
                         ++syncResult.stats.numIoExceptions;

diff --git a/src/com/cooliris/picasa/PicasaContentProvider.java b/src/com/cooliris/picasa/PicasaContentProvider.java
index fa61637..f9853ac 100644 (file)
--- a/src/com/cooliris/picasa/PicasaContentProvider.java
+++ b/src/com/cooliris/picasa/PicasaContentProvider.java
@@ -197,8 +197,6 @@ public final class PicasaContentProvider extends TableContentProvider {
     }
 
     private UserEntry[] syncUsers(SyncContext context, SyncResult syncResult) {
-        Log.i(TAG, "syncUsers");
-
         // Get authorized accounts.
         context.reloadAccounts();
         PicasaApi.AuthAccount[] accounts = context.accounts;
@@ -209,7 +207,6 @@ public final class PicasaContentProvider extends TableContentProvider {
         EntrySchema schema = UserEntry.SCHEMA;
         SQLiteDatabase db = context.db;
         Cursor cursor = schema.queryAll(db);
-        Log.i(TAG, "#users: " + cursor.getCount());
         if (cursor.moveToFirst()) {
             do {
                 // Read the current account.
@@ -220,10 +217,8 @@ public final class PicasaContentProvider extends TableContentProvider {
                 // not exist.
                 int i;
                 for (i = 0; i != numUsers; ++i) {
-                    Log.i(TAG, "Check " + accounts[i].user + " == " + entry.account);
                     if (accounts[i].user.equals(entry.account)) {
                         users[i] = entry;
-                        Log.e(TAG, "Updating user " + entry.account);
                         break;
                     }
                 }
@@ -245,7 +240,6 @@ public final class PicasaContentProvider extends TableContentProvider {
             if (entry == null) {
                 entry = new UserEntry();
                 entry.account = account.user;
-                Log.d(TAG, "insert/replace: " + schema.insertOrReplace(db, entry));
                 users[i] = entry;
                 Log.e(TAG, "Inserting user " + entry.account);
             }
@@ -351,7 +345,6 @@ public final class PicasaContentProvider extends TableContentProvider {
 
     private void syncAlbumPhotos(SyncContext context, final String account, AlbumEntry album, final SyncResult syncResult) {
         Log.i(TAG, "Syncing Picasa album: " + album.title);
-
         // Query existing album entry (id, dateEdited) sorted by ID.
         final SQLiteDatabase db = context.db;
         long albumId = album.id;

diff --git a/src/com/cooliris/picasa/PicasaSyncAdapter.java b/src/com/cooliris/picasa/PicasaSyncAdapter.java
index f507e6a..d6b4e67 100644 (file)
--- a/src/com/cooliris/picasa/PicasaSyncAdapter.java
+++ b/src/com/cooliris/picasa/PicasaSyncAdapter.java
@@ -23,7 +23,6 @@ public class PicasaSyncAdapter extends AbstractThreadedSyncAdapter {
     public void onPerformSync(Account account, Bundle extras, String authority, ContentProviderClient providerClient,
             SyncResult syncResult) {
         PicasaService.performSync(mContext, account, extras, syncResult);
-        Log.i(TAG, "SyncResult: " + syncResult.toDebugString());
     }
 
     public static final class AccountChangeReceiver extends BroadcastReceiver {