You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@usergrid.apache.org by sn...@apache.org on 2015/03/30 21:58:38 UTC

[34/50] incubator-usergrid git commit: Merge branch 'USERGRID-405' into USERGRID-448-appinfofix

Merge branch 'USERGRID-405' into USERGRID-448-appinfofix

Conflicts:
	stack/test-utils/src/main/java/org/apache/usergrid/setup/ConcurrentProcessSingleton.java


Project: http://git-wip-us.apache.org/repos/asf/incubator-usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-usergrid/commit/e47555e7
Tree: http://git-wip-us.apache.org/repos/asf/incubator-usergrid/tree/e47555e7
Diff: http://git-wip-us.apache.org/repos/asf/incubator-usergrid/diff/e47555e7

Branch: refs/heads/two-dot-o-dev
Commit: e47555e72c1c858b27787438f5ec3e5512298811
Parents: 68f56dd 9a391fe
Author: Dave Johnson <dm...@apigee.com>
Authored: Fri Mar 20 10:03:06 2015 -0400
Committer: Dave Johnson <dm...@apigee.com>
Committed: Fri Mar 20 10:03:06 2015 -0400

----------------------------------------------------------------------
 stack/core/pom.xml                              |  33 +-
 .../corepersistence/CpEntityManager.java        |   2 +-
 .../corepersistence/CpEntityManagerFactory.java |   7 +-
 .../corepersistence/CpRelationManager.java      |  24 +-
 .../usergrid/corepersistence/CpWalker.java      |  81 ++---
 .../events/EntityVersionDeletedHandler.java     |  72 ++--
 .../migration/EntityTypeMappingMigration.java   |  41 +--
 .../persistence/ObservableIterator.java         |   1 +
 .../migration/EntityTypeMappingMigrationIT.java |   2 +-
 .../PerformanceEntityRebuildIndexTest.java      |   3 +-
 stack/corepersistence/collection/pom.xml        |  14 +-
 .../impl/EntityCollectionManagerImpl.java       |  10 +-
 .../collection/impl/EntityDeletedTask.java      |  20 +-
 .../impl/EntityVersionCleanupTask.java          |  40 +--
 .../impl/EntityVersionCreatedTask.java          |  26 +-
 .../mvcc/stage/write/WriteCommit.java           |   2 +-
 .../mvcc/stage/write/WriteUniqueVerify.java     |   8 +-
 .../MvccEntitySerializationStrategyImpl.java    |  92 ++---
 .../MvccEntitySerializationStrategyV3Impl.java  |  94 ++---
 .../UniqueValueSerializationStrategyImpl.java   |   8 -
 .../migration/MvccEntityDataMigrationImpl.java  | 171 ++++------
 .../collection/util/EntityUtils.java            |  72 ----
 .../mvcc/stage/AbstractEntityStageTest.java     |   2 +-
 .../mvcc/stage/AbstractMvccEntityStageTest.java |   2 +-
 .../mvcc/stage/TestEntityGenerator.java         |   2 +-
 .../persistence/collection/rx/ParallelTest.java |  10 +-
 ...MvccEntitySerializationStrategyImplTest.java |   4 +-
 ...ccEntitySerializationStrategyV1ImplTest.java |   4 +-
 ...ccEntitySerializationStrategyV2ImplTest.java |   2 +-
 .../impl/SerializationComparison.java           |   4 +-
 ...ctMvccEntityDataMigrationV1ToV3ImplTest.java |   2 +-
 .../collection/util/InvalidEntityGenerator.java |   1 +
 stack/corepersistence/common/pom.xml            |  15 +-
 .../astyanax/MultiKeyColumnNameIterator.java    |   4 +-
 .../MultiKeyColumnNameIteratorTest.java         | 187 +++++-----
 .../astyanax/MultiRowColumnIteratorTest.java    |  50 ++-
 .../graph/impl/GraphManagerImpl.java            |   6 +-
 .../graph/impl/stage/EdgeMetaRepairImpl.java    |   2 +
 .../impl/stage/NodeDeleteListenerImpl.java      |   2 +-
 .../impl/migration/EdgeDataMigrationImpl.java   |  87 +++--
 .../persistence/graph/GraphManagerIT.java       |  17 +-
 .../graph/GraphManagerShardConsistencyIT.java   |   2 +-
 .../usergrid/persistence/graph/SimpleTest.java  |  12 +-
 .../migration/EdgeDataMigrationImplTest.java    |   2 +-
 stack/corepersistence/model/pom.xml             |   1 -
 .../persistence/model/util/EntityUtils.java     |  72 ++++
 stack/corepersistence/pom.xml                   |   8 +-
 stack/corepersistence/queryindex/pom.xml        |   6 -
 .../index/impl/EsEntityIndexImpl.java           |   2 +-
 .../persistence/index/query/EntityResults.java  | 108 ------
 .../persistence/index/query/Results.java        | 148 --------
 .../persistence/index/utils/ListUtils.java      |   6 +-
 .../persistence/index/guice/IndexTestFig.java   |  57 ++++
 .../index/guice/TestIndexModule.java            |   8 +-
 .../index/impl/CorePerformanceIT.java           | 339 -------------------
 .../impl/EntityConnectionIndexImplTest.java     | 306 -----------------
 .../persistence/index/impl/EntityIndexTest.java |  32 +-
 .../index/impl/IndexLoadTestsIT.java            | 138 ++++++++
 stack/pom.xml                                   |  14 +-
 stack/rest/pom.xml                              |   1 +
 stack/services/pom.xml                          |   1 +
 .../management/importer/ImportServiceImpl.java  |  34 +-
 .../impl/ApplicationQueueManagerImpl.java       | 195 +++++------
 .../setup/ConcurrentProcessSingleton.java       |  21 +-
 64 files changed, 908 insertions(+), 1831 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-usergrid/blob/e47555e7/stack/core/src/main/java/org/apache/usergrid/corepersistence/CpEntityManagerFactory.java
----------------------------------------------------------------------
diff --cc stack/core/src/main/java/org/apache/usergrid/corepersistence/CpEntityManagerFactory.java
index 3b47fa2,83c3d85..782b273
--- a/stack/core/src/main/java/org/apache/usergrid/corepersistence/CpEntityManagerFactory.java
+++ b/stack/core/src/main/java/org/apache/usergrid/corepersistence/CpEntityManagerFactory.java
@@@ -444,12 -468,12 +445,12 @@@ public class CpEntityManagerFactory imp
                  appScope.getApplication(),
                  scopeName);
  
 -            org.apache.usergrid.persistence.model.entity.Entity e =
 +            org.apache.usergrid.persistence.model.entity.Entity appInfo =
                      managerCache.getEntityCollectionManager( collScope ).load( targetId )
-                         .toBlockingObservable().lastOrDefault(null);
+                         .toBlocking().lastOrDefault(null);
  
 -            if ( e == null ) {
 -                logger.warn("Applicaion {} in index but not found in collections", targetId );
 +            if ( appInfo == null ) {
 +                logger.warn("Application {} in index but not found in collections", targetId );
                  continue;
              }
  

http://git-wip-us.apache.org/repos/asf/incubator-usergrid/blob/e47555e7/stack/core/src/test/java/org/apache/usergrid/persistence/PerformanceEntityRebuildIndexTest.java
----------------------------------------------------------------------
diff --cc stack/core/src/test/java/org/apache/usergrid/persistence/PerformanceEntityRebuildIndexTest.java
index 86b5c0f,52d4a48..8363a24
--- a/stack/core/src/test/java/org/apache/usergrid/persistence/PerformanceEntityRebuildIndexTest.java
+++ b/stack/core/src/test/java/org/apache/usergrid/persistence/PerformanceEntityRebuildIndexTest.java
@@@ -424,8 -427,7 +423,8 @@@ public class PerformanceEntityRebuildIn
          }
  
          if ( expectedEntities != -1 && expectedEntities != count ) {
 -            throw new RuntimeException("Did not get expected " + expectedEntities + " entities, instead got " + count );
 +            throw new RuntimeException("Did not get expected "
-                     + expectedEntities + " entities, instead got " + count );
++                + expectedEntities + " entities, instead got " + count );
          }
          return count;
      }

http://git-wip-us.apache.org/repos/asf/incubator-usergrid/blob/e47555e7/stack/corepersistence/queryindex/src/main/java/org/apache/usergrid/persistence/index/impl/EsEntityIndexImpl.java
----------------------------------------------------------------------

http://git-wip-us.apache.org/repos/asf/incubator-usergrid/blob/e47555e7/stack/services/src/main/java/org/apache/usergrid/management/importer/ImportServiceImpl.java
----------------------------------------------------------------------

http://git-wip-us.apache.org/repos/asf/incubator-usergrid/blob/e47555e7/stack/test-utils/src/main/java/org/apache/usergrid/setup/ConcurrentProcessSingleton.java
----------------------------------------------------------------------
diff --cc stack/test-utils/src/main/java/org/apache/usergrid/setup/ConcurrentProcessSingleton.java
index d8e124f,7cb5073..64a2f7a
--- a/stack/test-utils/src/main/java/org/apache/usergrid/setup/ConcurrentProcessSingleton.java
+++ b/stack/test-utils/src/main/java/org/apache/usergrid/setup/ConcurrentProcessSingleton.java
@@@ -103,22 -101,26 +100,32 @@@ public class ConcurrentProcessSingleto
  
                  // signal to other processes we've migrated, and they can proceed
                  barrier.proceed();
 -            }
  
 +                logger.info( "Waiting for setup to complete" );
 +                barrier.await( ONE_MINUTE );
 +                logger.info( "Setup to complete" );
 +
 +                lock.maybeReleaseLock();
  
 -            logger.info( "Waiting for setup to complete" );
 -            barrier.await( ONE_MINUTE );
 -            logger.info( "Setup to complete" );
 +            } else {
 +                throw new RuntimeException( "Unable to initialize system: could not get lock."
 +                    +" Some other process must be binding to port " + LOCK_PORT );
 +            }
  
-             // Commented out: Never release the lock, otherwise some other JVM may destroy the schema
-             // lock.maybeReleaseLock();
+             Runtime.getRuntime().addShutdownHook( new Thread(  ){
+                 @Override
+                 public void run() {
+                     try {
+                         lock.maybeReleaseLock();
+                     }
+                     catch ( IOException e ) {
+                         throw new RuntimeException( "Unable to release lock" );
+                     }
+                 }
+             });
+ 
          }
 +
          catch ( Exception e ) {
              throw new RuntimeException( "Unable to initialize system", e );
          }