Skip to content

Commit

Permalink
fix for test
Browse files Browse the repository at this point in the history
  • Loading branch information
Eric Schmidt committed Oct 24, 2022
1 parent 181b5ba commit e329b24
Showing 1 changed file with 8 additions and 3 deletions.
11 changes: 8 additions & 3 deletions bigtable/admin.go
Expand Up @@ -349,10 +349,15 @@ func (ac *AdminClient) updateTableWithConf(ctx context.Context, conf *UpdateTabl
}
lro, err := ac.tClient.UpdateTable(ctx, req)
if err != nil {
return err
return fmt.Errorf("error from update: %w", err)
}
// ignore the response table proto by passing in nil
return longrunning.InternalNewOperation(ac.lroClient, lro).Wait(ctx, nil)
var tbl btapb.Table
op := longrunning.InternalNewOperation(ac.lroClient, lro)
err = op.Wait(ctx, &tbl)
if err != nil {
return fmt.Errorf("error from operation: %v", err)
}
return nil
}

// DeleteTable deletes a table and all of its data.
Expand Down

0 comments on commit e329b24

Please sign in to comment.