Skip to content

Commit

Permalink
Remove unused lake/data.Object methods (#4792)
Browse files Browse the repository at this point in the history
StringRange formats zed.Values with %s.  Range formats them with %d.
Neither is very useful.
  • Loading branch information
nwt authored Oct 3, 2023
1 parent 7cde67c commit 7ee1443
Showing 1 changed file with 0 additions and 9 deletions.
9 changes: 0 additions & 9 deletions lake/data/object.go
Original file line number Diff line number Diff line change
Expand Up @@ -98,10 +98,6 @@ func plural(ordinal int) string {
return "s"
}

func (o Object) StringRange() string {
return fmt.Sprintf("%s %s %s", o.ID, o.Min, o.Max)
}

func (o *Object) Equal(to *Object) bool {
return o.ID == to.ID
}
Expand Down Expand Up @@ -145,11 +141,6 @@ func VectorURI(path *storage.URI, id ksuid.KSUID) *storage.URI {
return path.JoinPath(fmt.Sprintf("%s.vng", id))
}

func (o Object) Range() string {
//XXX need to handle any key... will the String method work?
return fmt.Sprintf("[%d-%d]", o.Min, o.Max)
}

// Remove deletes the row object and its seek index.
// Any 'not found' errors are ignored.
func (o Object) Remove(ctx context.Context, engine storage.Engine, path *storage.URI) error {
Expand Down

0 comments on commit 7ee1443

Please sign in to comment.