search

scylladb / gemini

Test data integrity by comparing against an Oracle running in parallel
Apache License 2.0
28 stars 16 forks source link

fix(stmtlogger): lets gemini work with empty 'test-statement-log-file' option #416

Closed illia-li closed 9 months ago

illia-li commented 9 months ago

Empty 'test-statement-log-file' option brings to panic. Reason:

func NewFileLogger(filename string) (StmtToFile, error) {
    if filename == "" {
        return nil, nil
...
func (fl *fileLogger) LogStmt(stmt *typedef.Stmt) {
    ch := fl.activeChannel.Load()
...