Don't abort on git.Blame errors

Signed-off-by: Juanjo Alvarez <[email protected]>

Author: Juanjo Alvarez

CHANGELOG.md

@@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Added BLAME function.
 ## Fixed
 - Removed redundant commit information from BLAME results.
+- Don't abort, just warn, on git.Blame errors.
 
 ## [0.24.0-rc2] - 2019-10-02
 

internal/function/blame.go

@@ -2,6 +2,9 @@ package function
 
 import (
 	"fmt"
+	"io"
+
+	"github.com/sirupsen/logrus"
 
 	"github.com/src-d/gitbase"
 	"github.com/src-d/go-mysql-server/sql"
@@ -17,10 +20,11 @@ type BlameGenerator struct {
 	curLine int
 	curFile *object.File
 	lines   []*git.Line
+	ctx     *sql.Context
 }
 
-func NewBlameGenerator(c *object.Commit, f *object.FileIter) (*BlameGenerator, error) {
-	return &BlameGenerator{commit: c, fIter: f, curLine: -1}, nil
+func NewBlameGenerator(c *object.Commit, f *object.FileIter, ctx *sql.Context) (*BlameGenerator, error) {
+	return &BlameGenerator{commit: c, fIter: f, curLine: -1, ctx: ctx}, nil
 }
 
 func (g *BlameGenerator) loadNewFile() error {
@@ -32,7 +36,11 @@ func (g *BlameGenerator) loadNewFile() error {
 
 	result, err := git.Blame(g.commit, g.curFile.Name)
 	if err != nil {
-		return err
+		msg := fmt.Sprintf("Error in BLAME for file %s: %s",
+			g.curFile.Name, err.Error())
+		logrus.Warn(msg)
+		g.ctx.Warn(0, msg)
+		return io.EOF
 	}
 
 	if len(result.Lines) == 0 {
@@ -145,7 +153,7 @@ func (b *Blame) Eval(ctx *sql.Context, row sql.Row) (interface{}, error) {
 		return nil, err
 	}
 
-	bg, err := NewBlameGenerator(commit, fIter)
+	bg, err := NewBlameGenerator(commit, fIter, ctx)
 	if err != nil {
 		return nil, err
 	}