ObjectReader: Allow getCommitGraph to throw IOException

ObjectReader#getCommitGraph doesn't report errors loading the
commit graph. The caller should be aware of the situation and
ultimately decide what to do.

Add IOException to ObjectReader#getCommitGraph signature. RevWalk
defaults to an empty commit-graph on IO errors.

Signed-off-by: Xing Huang <xingkhuang@google.com>
Change-Id: I38eeacff76c7f926b6dfb192d1e5916e40770024

1 files changed, 5 insertions, 2 deletions

diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/lib/ObjectReader.java b/org.eclipse.jgit/src/org/eclipse/jgit/lib/ObjectReader.java
index ae0cf42b12..69b2b5104e 100644
--- a/org.eclipse.jgit/src/org/eclipse/jgit/lib/ObjectReader.java
+++ b/org.eclipse.jgit/src/org/eclipse/jgit/lib/ObjectReader.java
@@ -512,9 +512,12 @@ public abstract class ObjectReader implements AutoCloseable {
 	 *         (default is
 	 *         {@value org.eclipse.jgit.lib.CoreConfig#DEFAULT_COMMIT_GRAPH_ENABLE}).
 	 *
+	 * @throws IOException
+	 *             if it cannot open any of the underlying commit graph.
+	 *
 	 * @since 6.5
 	 */
-	public Optional<CommitGraph> getCommitGraph() {
+	public Optional<CommitGraph> getCommitGraph() throws IOException {
 		return Optional.empty();
 	}
 
@@ -661,7 +664,7 @@ public abstract class ObjectReader implements AutoCloseable {
 		}
 
 		@Override
-		public Optional<CommitGraph> getCommitGraph() {
+		public Optional<CommitGraph> getCommitGraph() throws IOException{
 			return delegate().getCommitGraph();
 		}