You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by mr...@apache.org on 2018/04/26 08:06:14 UTC
svn commit: r1830171 - in /jackrabbit/oak/trunk/oak-store-document: pom.xml
src/main/java/org/apache/jackrabbit/oak/plugins/document/util/Utils.java
src/test/java/org/apache/jackrabbit/oak/plugins/document/util/UtilsTest.java
Author: mreutegg
Date: Thu Apr 26 08:06:14 2018
New Revision: 1830171
URL: http://svn.apache.org/viewvc?rev=1830171&view=rev
Log:
OAK-7442: Remove dependency to commons-codec
Modified:
jackrabbit/oak/trunk/oak-store-document/pom.xml
jackrabbit/oak/trunk/oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/util/Utils.java
jackrabbit/oak/trunk/oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/util/UtilsTest.java
Modified: jackrabbit/oak/trunk/oak-store-document/pom.xml
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-store-document/pom.xml?rev=1830171&r1=1830170&r2=1830171&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-store-document/pom.xml (original)
+++ jackrabbit/oak/trunk/oak-store-document/pom.xml Thu Apr 26 08:06:14 2018
@@ -129,10 +129,6 @@
<artifactId>guava</artifactId>
</dependency>
<dependency>
- <groupId>commons-codec</groupId>
- <artifactId>commons-codec</artifactId>
- </dependency>
- <dependency>
<groupId>commons-io</groupId>
<artifactId>commons-io</artifactId>
</dependency>
@@ -292,6 +288,11 @@
<scope>test</scope>
</dependency>
<dependency>
+ <groupId>commons-codec</groupId>
+ <artifactId>commons-codec</artifactId>
+ <scope>test</scope>
+ </dependency>
+ <dependency>
<groupId>org.apache.sling</groupId>
<artifactId>org.apache.sling.testing.osgi-mock</artifactId>
<scope>test</scope>
Modified: jackrabbit/oak/trunk/oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/util/Utils.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/util/Utils.java?rev=1830171&r1=1830170&r2=1830171&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/util/Utils.java (original)
+++ jackrabbit/oak/trunk/oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/util/Utils.java Thu Apr 26 08:06:14 2018
@@ -39,7 +39,6 @@ import com.google.common.base.Function;
import com.google.common.base.Predicate;
import com.google.common.collect.AbstractIterator;
-import org.apache.commons.codec.binary.Hex;
import org.apache.jackrabbit.oak.commons.OakVersion;
import org.apache.jackrabbit.oak.commons.PathUtils;
import org.apache.jackrabbit.oak.commons.StringUtils;
@@ -92,6 +91,8 @@ public class Utils {
private static final Charset UTF_8 = Charset.forName("UTF-8");
+ private static final char[] HEX_DIGITS = "0123456789abcdef".toCharArray();
+
/**
* A predicate for property and _deleted names.
*/
@@ -243,13 +244,33 @@ public class Utils {
String parent = PathUtils.getParentPath(path);
byte[] hash = digest.digest(parent.getBytes(UTF_8));
String name = PathUtils.getName(path);
- return depth + ":h" + Hex.encodeHexString(hash) + "/" + name;
+ StringBuilder sb = new StringBuilder(digest.getDigestLength() * 2 + name.length() + 6);
+ sb.append(depth).append(":h");
+ encodeHexString(hash, sb).append("/").append(name);
+ return sb.toString();
}
int depth = Utils.pathDepth(path);
return depth + ":" + path;
}
/**
+ * Encodes the given data as hexadecimal string representation and appends
+ * it to the {@code StringBuilder}. The hex digits are in lower case.
+ *
+ * @param data the bytes to encode.
+ * @param sb the hexadecimal string representation is appended to this
+ * {@code StringBuilder}.
+ * @return the {@code StringBuilder} passed to this method.
+ */
+ public static StringBuilder encodeHexString(byte[] data, StringBuilder sb) {
+ for (byte b : data) {
+ sb.append(HEX_DIGITS[(b >> 4) & 0xF]);
+ sb.append(HEX_DIGITS[(b & 0xF)]);
+ }
+ return sb;
+ }
+
+ /**
* Returns the parent id for given id if possible
*
* <p>It would return null in following cases
Modified: jackrabbit/oak/trunk/oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/util/UtilsTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/util/UtilsTest.java?rev=1830171&r1=1830170&r2=1830171&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/util/UtilsTest.java (original)
+++ jackrabbit/oak/trunk/oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/util/UtilsTest.java Thu Apr 26 08:06:14 2018
@@ -18,12 +18,14 @@ package org.apache.jackrabbit.oak.plugin
import java.util.Collections;
import java.util.List;
+import java.util.Random;
import java.util.concurrent.atomic.AtomicBoolean;
import com.google.common.base.Strings;
import com.google.common.collect.ImmutableList;
import com.google.common.collect.Iterables;
+import org.apache.commons.codec.binary.Hex;
import org.apache.jackrabbit.oak.api.CommitFailedException;
import org.apache.jackrabbit.oak.commons.PathUtils;
import org.apache.jackrabbit.oak.plugins.document.DocumentMK;
@@ -302,6 +304,19 @@ public class UtilsTest {
}
@Test
+ public void encodeHexString() {
+ Random r = new Random(42);
+ for (int i = 0; i < 1000; i++) {
+ int len = r.nextInt(100);
+ byte[] data = new byte[len];
+ r.nextBytes(data);
+ // compare against commons codec implementation
+ assertEquals(Hex.encodeHexString(data),
+ Utils.encodeHexString(data, new StringBuilder()).toString());
+ }
+ }
+
+ @Test
public void isLocalChange() {
RevisionVector empty = new RevisionVector();
Revision r11 = Revision.fromString("r1-0-1");