mirror of
https://github.com/Alfresco/alfresco-community-repo.git
synced 2025-09-10 14:11:58 +00:00
Compare commits
7 Commits
dependabot
...
repo-5439v
Author | SHA1 | Date | |
---|---|---|---|
|
019ad6da47 | ||
|
a17bb830f2 | ||
|
d062140883 | ||
|
993f42c877 | ||
|
6ba5c3ac35 | ||
|
a9cf423885 | ||
|
047c47def4 |
@@ -33,7 +33,7 @@ stages:
|
||||
- name: test
|
||||
if: commit_message !~ /\[skip tests\]/
|
||||
- name: release
|
||||
if: fork = false AND (branch = master OR branch =~ /release\/.*/) AND type != pull_request AND commit_message !~ /\[no release\]/
|
||||
if: fork = false AND (branch = master OR branch =~ /release\/.*/ OR branch =~/fix\/.*/) AND type != pull_request AND commit_message !~ /\[no release\]/
|
||||
- name: update_downstream
|
||||
if: fork = false AND (branch = master OR branch =~ /release\/.*/) AND type != pull_request AND commit_message !~ /\[no downstream\]/
|
||||
- name: trigger_downstream
|
||||
|
@@ -7,7 +7,7 @@
|
||||
<parent>
|
||||
<groupId>org.alfresco</groupId>
|
||||
<artifactId>alfresco-community-repo</artifactId>
|
||||
<version>8.393-SNAPSHOT</version>
|
||||
<version>repo-5439v2-c1</version>
|
||||
</parent>
|
||||
|
||||
<dependencies>
|
||||
|
@@ -424,6 +424,7 @@ public abstract class AbstractAsynchronouslyRefreshedCache<T>
|
||||
@Override
|
||||
public Void call()
|
||||
{
|
||||
liveLock.writeLock().lock();
|
||||
try
|
||||
{
|
||||
doCall();
|
||||
@@ -444,6 +445,10 @@ public abstract class AbstractAsynchronouslyRefreshedCache<T>
|
||||
}
|
||||
return null;
|
||||
}
|
||||
finally
|
||||
{
|
||||
liveLock.writeLock().unlock();
|
||||
}
|
||||
}
|
||||
|
||||
private void doCall() throws Exception
|
||||
|
@@ -7,7 +7,7 @@
|
||||
<parent>
|
||||
<groupId>org.alfresco</groupId>
|
||||
<artifactId>alfresco-community-repo</artifactId>
|
||||
<version>8.393-SNAPSHOT</version>
|
||||
<version>repo-5439v2-c1</version>
|
||||
</parent>
|
||||
|
||||
<properties>
|
||||
|
@@ -9,6 +9,6 @@
|
||||
<parent>
|
||||
<groupId>org.alfresco</groupId>
|
||||
<artifactId>alfresco-community-repo-packaging</artifactId>
|
||||
<version>8.393-SNAPSHOT</version>
|
||||
<version>repo-5439v2-c1</version>
|
||||
</parent>
|
||||
</project>
|
||||
|
@@ -7,7 +7,7 @@
|
||||
<parent>
|
||||
<groupId>org.alfresco</groupId>
|
||||
<artifactId>alfresco-community-repo-packaging</artifactId>
|
||||
<version>8.393-SNAPSHOT</version>
|
||||
<version>repo-5439v2-c1</version>
|
||||
</parent>
|
||||
|
||||
<properties>
|
||||
|
@@ -7,7 +7,7 @@
|
||||
<parent>
|
||||
<groupId>org.alfresco</groupId>
|
||||
<artifactId>alfresco-community-repo</artifactId>
|
||||
<version>8.393-SNAPSHOT</version>
|
||||
<version>repo-5439v2-c1</version>
|
||||
</parent>
|
||||
|
||||
<profiles>
|
||||
|
@@ -6,7 +6,7 @@
|
||||
<parent>
|
||||
<groupId>org.alfresco</groupId>
|
||||
<artifactId>alfresco-community-repo-packaging</artifactId>
|
||||
<version>8.393-SNAPSHOT</version>
|
||||
<version>repo-5439v2-c1</version>
|
||||
</parent>
|
||||
|
||||
<modules>
|
||||
|
@@ -9,7 +9,7 @@
|
||||
<parent>
|
||||
<groupId>org.alfresco</groupId>
|
||||
<artifactId>alfresco-community-repo-tests</artifactId>
|
||||
<version>8.393-SNAPSHOT</version>
|
||||
<version>repo-5439v2-c1</version>
|
||||
</parent>
|
||||
|
||||
<developers>
|
||||
|
@@ -9,7 +9,7 @@
|
||||
<parent>
|
||||
<groupId>org.alfresco</groupId>
|
||||
<artifactId>alfresco-community-repo-tests</artifactId>
|
||||
<version>8.393-SNAPSHOT</version>
|
||||
<version>repo-5439v2-c1</version>
|
||||
</parent>
|
||||
|
||||
<developers>
|
||||
|
@@ -9,7 +9,7 @@
|
||||
<parent>
|
||||
<groupId>org.alfresco</groupId>
|
||||
<artifactId>alfresco-community-repo-tests</artifactId>
|
||||
<version>8.393-SNAPSHOT</version>
|
||||
<version>repo-5439v2-c1</version>
|
||||
</parent>
|
||||
|
||||
<developers>
|
||||
|
@@ -9,7 +9,7 @@
|
||||
<parent>
|
||||
<groupId>org.alfresco</groupId>
|
||||
<artifactId>alfresco-community-repo-tests</artifactId>
|
||||
<version>8.393-SNAPSHOT</version>
|
||||
<version>repo-5439v2-c1</version>
|
||||
</parent>
|
||||
|
||||
<developers>
|
||||
|
@@ -9,7 +9,7 @@
|
||||
<parent>
|
||||
<groupId>org.alfresco</groupId>
|
||||
<artifactId>alfresco-community-repo-tests</artifactId>
|
||||
<version>8.393-SNAPSHOT</version>
|
||||
<version>repo-5439v2-c1</version>
|
||||
</parent>
|
||||
|
||||
<developers>
|
||||
|
@@ -7,7 +7,7 @@
|
||||
<parent>
|
||||
<groupId>org.alfresco</groupId>
|
||||
<artifactId>alfresco-community-repo-packaging</artifactId>
|
||||
<version>8.393-SNAPSHOT</version>
|
||||
<version>repo-5439v2-c1</version>
|
||||
</parent>
|
||||
|
||||
<properties>
|
||||
|
4
pom.xml
4
pom.xml
@@ -2,7 +2,7 @@
|
||||
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
|
||||
<modelVersion>4.0.0</modelVersion>
|
||||
<artifactId>alfresco-community-repo</artifactId>
|
||||
<version>8.393-SNAPSHOT</version>
|
||||
<version>repo-5439v2-c1</version>
|
||||
<packaging>pom</packaging>
|
||||
<name>Alfresco Community Repo Parent</name>
|
||||
|
||||
@@ -116,7 +116,7 @@
|
||||
<connection>scm:git:https://github.com/Alfresco/alfresco-community-repo.git</connection>
|
||||
<developerConnection>scm:git:https://github.com/Alfresco/alfresco-community-repo.git</developerConnection>
|
||||
<url>https://github.com/Alfresco/alfresco-community-repo</url>
|
||||
<tag>HEAD</tag>
|
||||
<tag>repo-5439v2-c1</tag>
|
||||
</scm>
|
||||
|
||||
<distributionManagement>
|
||||
|
@@ -7,7 +7,7 @@
|
||||
<parent>
|
||||
<groupId>org.alfresco</groupId>
|
||||
<artifactId>alfresco-community-repo</artifactId>
|
||||
<version>8.393-SNAPSHOT</version>
|
||||
<version>repo-5439v2-c1</version>
|
||||
</parent>
|
||||
|
||||
<dependencies>
|
||||
|
@@ -7,7 +7,7 @@
|
||||
<parent>
|
||||
<groupId>org.alfresco</groupId>
|
||||
<artifactId>alfresco-community-repo</artifactId>
|
||||
<version>8.393-SNAPSHOT</version>
|
||||
<version>repo-5439v2-c1</version>
|
||||
</parent>
|
||||
|
||||
<dependencies>
|
||||
|
@@ -150,24 +150,10 @@ public final class DefaultSimpleCache<K extends Serializable, V extends Object>
|
||||
* @return <code>true</code> if the put resulted in a change in value, <code>false</code> otherwise.
|
||||
*/
|
||||
public boolean putAndCheckUpdate(K key, V value)
|
||||
{
|
||||
return putAndCheckUpdate(key, value, false);
|
||||
}
|
||||
|
||||
/**
|
||||
* <code>put</code> method that may be used to check for updates in a thread-safe manner.
|
||||
*
|
||||
* @param includeNewCheck if true then we include the new value in the check
|
||||
* @return <code>true</code> if the put resulted in a change in value,
|
||||
* or if includeNewCheck is true and the put resulted in a new value,
|
||||
* <code>false</code> otherwise.
|
||||
*/
|
||||
public boolean putAndCheckUpdate(K key, V value, boolean includeNewCheck)
|
||||
{
|
||||
AbstractMap.SimpleImmutableEntry<K, V> kvp = new AbstractMap.SimpleImmutableEntry<K, V>(key, value);
|
||||
AbstractMap.SimpleImmutableEntry<K, V> priorKVP = cache.asMap().put(key, kvp);
|
||||
|
||||
return (includeNewCheck && priorKVP == null) || (priorKVP != null && (!priorKVP.equals(kvp)));
|
||||
return (priorKVP != null && (!priorKVP.equals(kvp)));
|
||||
}
|
||||
|
||||
@Override
|
||||
|
@@ -136,7 +136,6 @@
|
||||
<property name="tenantAware" value="false" />
|
||||
<property name="cacheStats" ref="cacheStatistics"/>
|
||||
<property name="cacheStatsEnabled" value="${cache.immutableEntitySharedCache.tx.statsEnabled}"/>
|
||||
<property name="allowEqualsChecks" value="true" />
|
||||
</bean>
|
||||
|
||||
|
||||
|
@@ -131,45 +131,6 @@ public class DefaultSimpleCacheTest extends SimpleCacheTestBase<DefaultSimpleCac
|
||||
assertEquals(false, cache.putAndCheckUpdate(104, "104"));
|
||||
assertEquals(true, cache.putAndCheckUpdate(104, null));
|
||||
}
|
||||
|
||||
@Test
|
||||
public void putAndCheckUpdateIncludeNewCheck()
|
||||
{
|
||||
// Put an initial value
|
||||
cache.put(101, "101");
|
||||
// Update it
|
||||
assertEquals(true, cache.putAndCheckUpdate(101, "99101", true));
|
||||
// Check the value really was updated
|
||||
assertEquals("99101", cache.get(101));
|
||||
|
||||
// Precondition: no value for key 102
|
||||
assertFalse(cache.contains(102));
|
||||
// Put a value - and test the return
|
||||
assertEquals(true, cache.putAndCheckUpdate(102, "102", true));
|
||||
assertEquals("102", cache.get(102));
|
||||
|
||||
cache.put(103, null);
|
||||
assertEquals(true, cache.putAndCheckUpdate(103, "103", true));
|
||||
// Repeat the put, this should not be an update
|
||||
assertEquals(false, cache.putAndCheckUpdate(103, "103", true));
|
||||
|
||||
assertFalse(cache.contains(104));
|
||||
assertEquals(true, cache.putAndCheckUpdate(104, null, true));
|
||||
// Repeat putting null - still not an update, as we had that value a moment ago.
|
||||
assertEquals(false, cache.putAndCheckUpdate(104, null, true));
|
||||
// Now an update
|
||||
assertEquals(true, cache.putAndCheckUpdate(104, "104", true));
|
||||
// Another update
|
||||
assertEquals(true, cache.putAndCheckUpdate(104, "99104", true));
|
||||
// Another update, back to null
|
||||
assertEquals(true, cache.putAndCheckUpdate(104, null, true));
|
||||
// Not an update - still null
|
||||
assertEquals(false, cache.putAndCheckUpdate(104, null, true));
|
||||
|
||||
cache.remove(104);
|
||||
assertEquals(true, cache.putAndCheckUpdate(104, "104", true));
|
||||
assertEquals(true, cache.putAndCheckUpdate(104, null, true));
|
||||
}
|
||||
|
||||
// TODO: Timer-based tests are not ideal. An alternative approach is to factor out the CacheBuilder.newBuilder()
|
||||
// call to a protected method, override that in this test class to return a mock and use the mock to check
|
||||
|
@@ -14,6 +14,7 @@ git config user.email "${GIT_EMAIL}"
|
||||
mvn -B \
|
||||
-PfullBuild,all-tas-tests \
|
||||
"-Darguments=-PfullBuild,all-tas-tests -DskipTests -Dbuild-number=${TRAVIS_BUILD_NUMBER}" \
|
||||
-DreleaseVersion=repo-5439v2-c1 -DdevelopmentVersion=8.393-SNAPSHOT \
|
||||
release:clean release:prepare release:perform \
|
||||
-DscmCommentPrefix="[maven-release-plugin][skip ci] " \
|
||||
-Dusername="${GIT_USERNAME}" \
|
||||
|
Reference in New Issue
Block a user