fix(ds.set_body): don't assign schemas when using set_body

ds.set_body is currently always assigning ds.Structure whenever it's called, which is trampling schemas. instead set_body should _never_ set a schema.
qri-io · Mar 5, 2019 · df4742b · df4742b
1 parent 9f06ef9
commit df4742b
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 5 deletions.
diff --git a/ds/dataset.go b/ds/dataset.go
@@ -206,9 +206,6 @@ func (d *Dataset) SetBody(thread *starlark.Thread, _ *starlark.Builtin, args sta
 		Schema: sch,
 	}
 
-	if d.ds.Structure == nil {
-		d.ds.Structure = st
-	}
 	w, err := dsio.NewEntryBuffer(st)
 	if err != nil {
 		return starlark.None, err

diff --git a/transform.go b/transform.go
@@ -116,8 +116,6 @@ func ExecScript(ds *dataset.Dataset, opts ...func(o *ExecOpts)) error {
 	tr := io.TeeReader(script, buf)
 	pipeScript := qfs.NewMemfileReader(script.FileName(), tr)
 
-	// buffer for script output
-
 	t := &transform{
 		node:      o.Node,
 		ds:        ds,