Merge branch 'stable-5.5'
* stable-5.5: WorkingTreeIterator: handle different timestamp resolutions Change-Id: I61df7392d8826ab526c39da2a8718723a1cc602a Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>
This commit is contained in:
commit
7be05205dd
|
@ -0,0 +1,182 @@
|
||||||
|
/*
|
||||||
|
* Copyright (C) 2019, Thomas Wolf <thomas.wolf@paranor.ch>
|
||||||
|
* and other copyright owners as documented in the project's IP log.
|
||||||
|
*
|
||||||
|
* This program and the accompanying materials are made available
|
||||||
|
* under the terms of the Eclipse Distribution License v1.0 which
|
||||||
|
* accompanies this distribution, is reproduced below, and is
|
||||||
|
* available at http://www.eclipse.org/org/documents/edl-v10.php
|
||||||
|
*
|
||||||
|
* All rights reserved.
|
||||||
|
*
|
||||||
|
* Redistribution and use in source and binary forms, with or
|
||||||
|
* without modification, are permitted provided that the following
|
||||||
|
* conditions are met:
|
||||||
|
*
|
||||||
|
* - Redistributions of source code must retain the above copyright
|
||||||
|
* notice, this list of conditions and the following disclaimer.
|
||||||
|
*
|
||||||
|
* - Redistributions in binary form must reproduce the above
|
||||||
|
* copyright notice, this list of conditions and the following
|
||||||
|
* disclaimer in the documentation and/or other materials provided
|
||||||
|
* with the distribution.
|
||||||
|
*
|
||||||
|
* - Neither the name of the Eclipse Foundation, Inc. nor the
|
||||||
|
* names of its contributors may be used to endorse or promote
|
||||||
|
* products derived from this software without specific prior
|
||||||
|
* written permission.
|
||||||
|
*
|
||||||
|
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND
|
||||||
|
* CONTRIBUTORS "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES,
|
||||||
|
* INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
|
||||||
|
* OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
|
||||||
|
* ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR
|
||||||
|
* CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
|
||||||
|
* SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
|
||||||
|
* NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
|
||||||
|
* LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
|
||||||
|
* CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
|
||||||
|
* STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
|
||||||
|
* ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
|
||||||
|
* ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
|
||||||
|
*/
|
||||||
|
package org.eclipse.jgit.treewalk;
|
||||||
|
|
||||||
|
import static org.junit.Assert.assertEquals;
|
||||||
|
import static org.junit.Assert.assertTrue;
|
||||||
|
|
||||||
|
import java.time.Instant;
|
||||||
|
|
||||||
|
import org.junit.Test;
|
||||||
|
|
||||||
|
public class InstantComparatorTest {
|
||||||
|
|
||||||
|
private final InstantComparator cmp = new InstantComparator();
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void compareNow() {
|
||||||
|
Instant now = Instant.now();
|
||||||
|
assertEquals(0, cmp.compare(now, now));
|
||||||
|
assertEquals(0, cmp.compare(now, now, true));
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void compareSeconds() {
|
||||||
|
Instant now = Instant.now();
|
||||||
|
Instant t = Instant.ofEpochSecond(now.getEpochSecond());
|
||||||
|
Instant s = Instant.ofEpochSecond(now.getEpochSecond(), 123456789);
|
||||||
|
assertEquals(0, cmp.compare(t, s));
|
||||||
|
assertEquals(0, cmp.compare(t, t));
|
||||||
|
assertEquals(0, cmp.compare(s, t));
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void compareSecondsOnly() {
|
||||||
|
Instant now = Instant.now();
|
||||||
|
Instant t = Instant.ofEpochSecond(now.getEpochSecond(), 987654321);
|
||||||
|
Instant s = Instant.ofEpochSecond(now.getEpochSecond(), 123456789);
|
||||||
|
assertEquals(0, cmp.compare(t, s, true));
|
||||||
|
assertEquals(0, cmp.compare(t, t, true));
|
||||||
|
assertEquals(0, cmp.compare(s, t, true));
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void compareSecondsUnequal() {
|
||||||
|
Instant now = Instant.now();
|
||||||
|
Instant t = Instant.ofEpochSecond(now.getEpochSecond());
|
||||||
|
Instant s = Instant.ofEpochSecond(now.getEpochSecond() - 1L);
|
||||||
|
assertTrue(cmp.compare(s, t) < 0);
|
||||||
|
assertTrue(cmp.compare(t, s) > 0);
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void compareMillisEqual() {
|
||||||
|
Instant now = Instant.now();
|
||||||
|
Instant t = Instant.ofEpochSecond(now.getEpochSecond(), 123000000);
|
||||||
|
Instant s = Instant.ofEpochSecond(now.getEpochSecond(), 123456789);
|
||||||
|
assertEquals(0, cmp.compare(s, t));
|
||||||
|
assertEquals(0, cmp.compare(t, t));
|
||||||
|
assertEquals(0, cmp.compare(t, s));
|
||||||
|
s = Instant.ofEpochSecond(now.getEpochSecond(), 123456000);
|
||||||
|
assertEquals(0, cmp.compare(s, t));
|
||||||
|
assertEquals(0, cmp.compare(t, s));
|
||||||
|
s = Instant.ofEpochSecond(now.getEpochSecond(), 123400000);
|
||||||
|
assertEquals(0, cmp.compare(s, t));
|
||||||
|
assertEquals(0, cmp.compare(t, s));
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void compareMillisUnequal() {
|
||||||
|
Instant now = Instant.now();
|
||||||
|
Instant t = Instant.ofEpochSecond(now.getEpochSecond(), 123000000);
|
||||||
|
Instant s = Instant.ofEpochSecond(now.getEpochSecond(), 122000000);
|
||||||
|
assertTrue(cmp.compare(s, t) < 0);
|
||||||
|
assertTrue(cmp.compare(t, s) > 0);
|
||||||
|
t = Instant.ofEpochSecond(now.getEpochSecond(), 130000000);
|
||||||
|
assertTrue(cmp.compare(s, t) < 0);
|
||||||
|
assertTrue(cmp.compare(t, s) > 0);
|
||||||
|
t = Instant.ofEpochSecond(now.getEpochSecond(), 200000000);
|
||||||
|
assertTrue(cmp.compare(s, t) < 0);
|
||||||
|
assertTrue(cmp.compare(t, s) > 0);
|
||||||
|
s = Instant.ofEpochSecond(now.getEpochSecond() - 1L, 123000000);
|
||||||
|
assertTrue(cmp.compare(s, t) < 0);
|
||||||
|
assertTrue(cmp.compare(t, s) > 0);
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void compareMicrosEqual() {
|
||||||
|
Instant now = Instant.now();
|
||||||
|
Instant t = Instant.ofEpochSecond(now.getEpochSecond(), 123456000);
|
||||||
|
Instant s = Instant.ofEpochSecond(now.getEpochSecond(), 123456789);
|
||||||
|
assertEquals(0, cmp.compare(s, t));
|
||||||
|
assertEquals(0, cmp.compare(t, s));
|
||||||
|
s = Instant.ofEpochSecond(now.getEpochSecond(), 123456700);
|
||||||
|
assertEquals(0, cmp.compare(s, t));
|
||||||
|
assertEquals(0, cmp.compare(t, s));
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void compareMicrosUnequal() {
|
||||||
|
Instant now = Instant.now();
|
||||||
|
Instant t = Instant.ofEpochSecond(now.getEpochSecond(), 123456000);
|
||||||
|
Instant s = Instant.ofEpochSecond(now.getEpochSecond(), 123455000);
|
||||||
|
assertTrue(cmp.compare(s, t) < 0);
|
||||||
|
assertTrue(cmp.compare(t, s) > 0);
|
||||||
|
t = Instant.ofEpochSecond(now.getEpochSecond(), 123460000);
|
||||||
|
assertTrue(cmp.compare(s, t) < 0);
|
||||||
|
assertTrue(cmp.compare(t, s) > 0);
|
||||||
|
t = Instant.ofEpochSecond(now.getEpochSecond(), 123500000);
|
||||||
|
assertTrue(cmp.compare(s, t) < 0);
|
||||||
|
assertTrue(cmp.compare(t, s) > 0);
|
||||||
|
s = Instant.ofEpochSecond(now.getEpochSecond() - 1L, 123456000);
|
||||||
|
assertTrue(cmp.compare(s, t) < 0);
|
||||||
|
assertTrue(cmp.compare(t, s) > 0);
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void compareNanosEqual() {
|
||||||
|
Instant now = Instant.now();
|
||||||
|
Instant t = Instant.ofEpochSecond(now.getEpochSecond(), 123456789);
|
||||||
|
Instant s = Instant.ofEpochSecond(now.getEpochSecond(), 123456789);
|
||||||
|
assertEquals(0, cmp.compare(s, t));
|
||||||
|
assertEquals(0, cmp.compare(t, s));
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void compareNanosUnequal() {
|
||||||
|
Instant now = Instant.now();
|
||||||
|
Instant t = Instant.ofEpochSecond(now.getEpochSecond(), 123456789);
|
||||||
|
Instant s = Instant.ofEpochSecond(now.getEpochSecond(), 123456700);
|
||||||
|
assertTrue(cmp.compare(s, t) < 0);
|
||||||
|
assertTrue(cmp.compare(t, s) > 0);
|
||||||
|
t = Instant.ofEpochSecond(now.getEpochSecond(), 123456800);
|
||||||
|
assertTrue(cmp.compare(s, t) < 0);
|
||||||
|
assertTrue(cmp.compare(t, s) > 0);
|
||||||
|
s = Instant.ofEpochSecond(now.getEpochSecond() - 1L, 123456789);
|
||||||
|
assertTrue(cmp.compare(s, t) < 0);
|
||||||
|
assertTrue(cmp.compare(t, s) > 0);
|
||||||
|
s = Instant.ofEpochSecond(now.getEpochSecond(), 123456788);
|
||||||
|
assertTrue(cmp.compare(s, t) < 0);
|
||||||
|
assertTrue(cmp.compare(t, s) > 0);
|
||||||
|
}
|
||||||
|
}
|
|
@ -0,0 +1,112 @@
|
||||||
|
/*
|
||||||
|
* Copyright (C) 2019, Thomas Wolf <thomas.wolf@paranor.ch>
|
||||||
|
* and other copyright owners as documented in the project's IP log.
|
||||||
|
*
|
||||||
|
* This program and the accompanying materials are made available
|
||||||
|
* under the terms of the Eclipse Distribution License v1.0 which
|
||||||
|
* accompanies this distribution, is reproduced below, and is
|
||||||
|
* available at http://www.eclipse.org/org/documents/edl-v10.php
|
||||||
|
*
|
||||||
|
* All rights reserved.
|
||||||
|
*
|
||||||
|
* Redistribution and use in source and binary forms, with or
|
||||||
|
* without modification, are permitted provided that the following
|
||||||
|
* conditions are met:
|
||||||
|
*
|
||||||
|
* - Redistributions of source code must retain the above copyright
|
||||||
|
* notice, this list of conditions and the following disclaimer.
|
||||||
|
*
|
||||||
|
* - Redistributions in binary form must reproduce the above
|
||||||
|
* copyright notice, this list of conditions and the following
|
||||||
|
* disclaimer in the documentation and/or other materials provided
|
||||||
|
* with the distribution.
|
||||||
|
*
|
||||||
|
* - Neither the name of the Eclipse Foundation, Inc. nor the
|
||||||
|
* names of its contributors may be used to endorse or promote
|
||||||
|
* products derived from this software without specific prior
|
||||||
|
* written permission.
|
||||||
|
*
|
||||||
|
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND
|
||||||
|
* CONTRIBUTORS "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES,
|
||||||
|
* INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
|
||||||
|
* OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
|
||||||
|
* ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR
|
||||||
|
* CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
|
||||||
|
* SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
|
||||||
|
* NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
|
||||||
|
* LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
|
||||||
|
* CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
|
||||||
|
* STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
|
||||||
|
* ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
|
||||||
|
* ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
|
||||||
|
*/
|
||||||
|
package org.eclipse.jgit.treewalk;
|
||||||
|
|
||||||
|
import java.time.Instant;
|
||||||
|
import java.util.Comparator;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Specialized comparator for {@link Instant}s. If either timestamp has a zero
|
||||||
|
* fraction, compares only seconds. If either timestamp has no time fraction
|
||||||
|
* smaller than a millisecond, compares only milliseconds. If either timestamp
|
||||||
|
* has no fraction smaller than a microsecond, compares only microseconds.
|
||||||
|
*/
|
||||||
|
class InstantComparator implements Comparator<Instant> {
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public int compare(Instant a, Instant b) {
|
||||||
|
return compare(a, b, false);
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Compares two {@link Instant}s to the lower resolution of the two
|
||||||
|
* instants. See {@link InstantComparator}.
|
||||||
|
*
|
||||||
|
* @param a
|
||||||
|
* first {@link Instant} to compare
|
||||||
|
* @param b
|
||||||
|
* second {@link Instant} to compare
|
||||||
|
* @param forceSecondsOnly
|
||||||
|
* whether to omit all fraction comparison
|
||||||
|
* @return a value < 0 if a < b, a value > 0 if a > b, and 0 if
|
||||||
|
* a == b
|
||||||
|
*/
|
||||||
|
public int compare(Instant a, Instant b, boolean forceSecondsOnly) {
|
||||||
|
long aSeconds = a.getEpochSecond();
|
||||||
|
long bSeconds = b.getEpochSecond();
|
||||||
|
int result = Long.compare(aSeconds, bSeconds);
|
||||||
|
if (result != 0) {
|
||||||
|
return result;
|
||||||
|
}
|
||||||
|
int aSubSecond = a.getNano();
|
||||||
|
int bSubSecond = b.getNano();
|
||||||
|
if (forceSecondsOnly || (aSubSecond == 0)
|
||||||
|
|| (bSubSecond == 0)) {
|
||||||
|
// Don't check the subseconds part.
|
||||||
|
return 0;
|
||||||
|
} else if (aSubSecond != bSubSecond) {
|
||||||
|
// If either has nothing smaller than a millisecond, compare only
|
||||||
|
// milliseconds.
|
||||||
|
int aSubMillis = aSubSecond % 1_000_000;
|
||||||
|
int bSubMillis = bSubSecond % 1_000_000;
|
||||||
|
if (aSubMillis == 0) {
|
||||||
|
bSubSecond -= bSubMillis;
|
||||||
|
} else if (bSubMillis == 0) {
|
||||||
|
aSubSecond -= aSubMillis;
|
||||||
|
} else {
|
||||||
|
// Same again, but for microsecond resolution. NTFS has 100ns
|
||||||
|
// resolution, but WindowsFileAttributes may provide only
|
||||||
|
// microseconds (1000ns). Similar for some Unix file systems.
|
||||||
|
int aSubMicros = aSubSecond % 1000;
|
||||||
|
int bSubMicros = bSubSecond % 1000;
|
||||||
|
if (aSubMicros == 0) {
|
||||||
|
bSubSecond -= bSubMicros;
|
||||||
|
} else if (bSubMicros == 0) {
|
||||||
|
aSubSecond -= aSubMicros;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return Integer.compare(aSubSecond, bSubSecond);
|
||||||
|
}
|
||||||
|
|
||||||
|
}
|
|
@ -174,6 +174,9 @@ public abstract class WorkingTreeIterator extends AbstractTreeIterator {
|
||||||
/** The offset of the content id in {@link #idBuffer()} */
|
/** The offset of the content id in {@link #idBuffer()} */
|
||||||
private int contentIdOffset;
|
private int contentIdOffset;
|
||||||
|
|
||||||
|
/** A comparator for {@link Instant}s. */
|
||||||
|
private final InstantComparator timestampComparator = new InstantComparator();
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Create a new iterator with no parent.
|
* Create a new iterator with no parent.
|
||||||
*
|
*
|
||||||
|
@ -931,27 +934,19 @@ public MetadataDiff compareMetadata(DirCacheEntry entry) {
|
||||||
if (!entry.isSmudged() && entry.getLength() != (int) getEntryLength())
|
if (!entry.isSmudged() && entry.getLength() != (int) getEntryLength())
|
||||||
return MetadataDiff.DIFFER_BY_METADATA;
|
return MetadataDiff.DIFFER_BY_METADATA;
|
||||||
|
|
||||||
// Git under windows only stores seconds so we round the timestamp
|
// Cache and file timestamps may differ in resolution. Therefore don't
|
||||||
// Java gives us if it looks like the timestamp in index is seconds
|
// compare instants directly but use a comparator that compares only
|
||||||
// only. Otherwise we compare the timestamp at nanosecond precision,
|
// up to the lower apparent resolution of either timestamp.
|
||||||
// unless core.checkstat is set to "minimal", in which case we only
|
//
|
||||||
// compare the whole second part.
|
// If core.checkstat is set to "minimal", compare only the seconds part.
|
||||||
Instant cacheLastModified = entry.getLastModifiedInstant();
|
Instant cacheLastModified = entry.getLastModifiedInstant();
|
||||||
Instant fileLastModified = getEntryLastModifiedInstant();
|
Instant fileLastModified = getEntryLastModifiedInstant();
|
||||||
if ((getOptions().getCheckStat() == CheckStat.MINIMAL)
|
if (timestampComparator.compare(cacheLastModified, fileLastModified,
|
||||||
|| (cacheLastModified.getNano() == 0)
|
getOptions().getCheckStat() == CheckStat.MINIMAL) != 0) {
|
||||||
// Some Java version on Linux return whole seconds only even
|
|
||||||
// when the file systems supports more precision.
|
|
||||||
|| (fileLastModified.getNano() == 0)) {
|
|
||||||
if (fileLastModified.getEpochSecond() != cacheLastModified
|
|
||||||
.getEpochSecond()) {
|
|
||||||
return MetadataDiff.DIFFER_BY_TIMESTAMP;
|
|
||||||
} else if (entry.isSmudged()) {
|
|
||||||
return MetadataDiff.SMUDGED;
|
|
||||||
}
|
|
||||||
} else if (!fileLastModified.equals(cacheLastModified)) {
|
|
||||||
return MetadataDiff.DIFFER_BY_TIMESTAMP;
|
return MetadataDiff.DIFFER_BY_TIMESTAMP;
|
||||||
} else if (entry.isSmudged()) {
|
}
|
||||||
|
|
||||||
|
if (entry.isSmudged()) {
|
||||||
return MetadataDiff.SMUDGED;
|
return MetadataDiff.SMUDGED;
|
||||||
}
|
}
|
||||||
// The file is clean when when comparing timestamps
|
// The file is clean when when comparing timestamps
|
||||||
|
|
Loading…
Reference in New Issue