From 1eecc82cec2409ed3b69cf96f3206a2fb6c7b9f1 Mon Sep 17 00:00:00 2001
From: "Shawn O. Pearce" <spearce@spearce.org>
Date: Fri, 24 Jun 2011 12:55:19 -0700
Subject: [PATCH] Improve performance when writing trees and small blobs

ObjectDirectoryInserter was always creating a temporary file,
writing the complete compressed contents of a tree, fsync()'ing
that to stable storage, and only then checking to see if there
was already an object with the same SHA-1 in the repository.

For commits this strategy makes some sense, the commit is very
unlikely to exist in the repository, as there are embedded times
and these change with each commit.

However for trees coming out of DirCache, it is more common for the
tree to already exist in the repository. Most subdirectories are
not modified in any given commit.  Doing all of this local file IO
for things that already exist is very slow.

Try to detect cases where the object is "small enough" that it can
be processed entirely in memory, and avoid doing disk IO entirely
if the object already exists.

Also increase the size of the output buffer for the deflation.
This should boost the average write(2) syscall size from 512 bytes
to 8192 bytes, making streaming of large compressed contents to
disk slightly more efficient.

Change-Id: I1d40364e8725468522435814631916d73174c92b
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
---
 .../storage/file/ObjectDirectoryInserter.java | 60 +++++++++++++++++--
 1 file changed, 56 insertions(+), 4 deletions(-)

diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/ObjectDirectoryInserter.java b/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/ObjectDirectoryInserter.java
index 0e46b6e14..77902dfb4 100644
--- a/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/ObjectDirectoryInserter.java
+++ b/org.eclipse.jgit/src/org/eclipse/jgit/storage/file/ObjectDirectoryInserter.java
@@ -65,6 +65,7 @@ import org.eclipse.jgit.lib.ObjectId;
 import org.eclipse.jgit.lib.ObjectInserter;
 import org.eclipse.jgit.transport.PackParser;
 import org.eclipse.jgit.util.FileUtils;
+import org.eclipse.jgit.util.IO;
 
 /** Creates loose objects in a {@link ObjectDirectory}. */
 class ObjectDirectoryInserter extends ObjectInserter {
@@ -79,13 +80,36 @@ class ObjectDirectoryInserter extends ObjectInserter {
 		config = cfg.get(WriteConfig.KEY);
 	}
 
+	@Override
+	public ObjectId insert(int type, byte[] data, int off, int len)
+			throws IOException {
+		ObjectId id = idFor(type, data, off, len);
+		if (db.has(id)) {
+			return id;
+		} else {
+			File tmp = toTemp(type, data, off, len);
+			return insertOneObject(tmp, id);
+		}
+	}
+
 	@Override
 	public ObjectId insert(final int type, long len, final InputStream is)
 			throws IOException {
-		final MessageDigest md = digest();
-		final File tmp = toTemp(md, type, len, is);
-		final ObjectId id = ObjectId.fromRaw(md.digest());
+		if (len <= buffer().length) {
+			byte[] buf = buffer();
+			IO.readFully(is, buf, 0, (int) len);
+			return insert(type, buf, 0, (int) len);
+
+		} else {
+			MessageDigest md = digest();
+			File tmp = toTemp(md, type, len, is);
+			ObjectId id = ObjectId.fromRaw(md.digest());
+			return insertOneObject(tmp, id);
+		}
+	}
 
+	private ObjectId insertOneObject(final File tmp, final ObjectId id)
+			throws IOException, ObjectWritingException {
 		switch (db.insertUnpackedObject(tmp, id, false /* no duplicate */)) {
 		case INSERTED:
 		case EXISTS_PACKED:
@@ -161,6 +185,34 @@ class ObjectDirectoryInserter extends ObjectInserter {
 		}
 	}
 
+	private File toTemp(final int type, final byte[] buf, final int pos,
+			final int len) throws IOException, FileNotFoundException {
+		boolean delete = true;
+		File tmp = newTempFile();
+		try {
+			FileOutputStream fOut = new FileOutputStream(tmp);
+			try {
+				OutputStream out = fOut;
+				if (config.getFSyncObjectFiles())
+					out = Channels.newOutputStream(fOut.getChannel());
+				DeflaterOutputStream cOut = compress(out);
+				writeHeader(cOut, type, len);
+				cOut.write(buf, pos, len);
+				cOut.finish();
+			} finally {
+				if (config.getFSyncObjectFiles())
+					fOut.getChannel().force(true);
+				fOut.close();
+			}
+
+			delete = false;
+			return tmp;
+		} finally {
+			if (delete)
+				FileUtils.delete(tmp);
+		}
+	}
+
 	void writeHeader(OutputStream out, final int type, long len)
 			throws IOException {
 		out.write(Constants.encodedTypeString(type));
@@ -178,7 +230,7 @@ class ObjectDirectoryInserter extends ObjectInserter {
 			deflate = new Deflater(config.getCompression());
 		else
 			deflate.reset();
-		return new DeflaterOutputStream(out, deflate);
+		return new DeflaterOutputStream(out, deflate, 8192);
 	}
 
 	private static EOFException shortInput(long missing) {