Browse Source

fix several minor golint style issues

pull/1624/head
Dmitry Savintsev 9 years ago
parent
commit
7fdb62c253
  1. 4
      notifier/notifier.go
  2. 3
      promql/lex.go
  3. 21
      promql/parse.go
  4. 2
      storage/remote/graphite/client.go

4
notifier/notifier.go

@ -42,7 +42,7 @@ const (
subsystem = "notifications"
)
// Handler is responsible for dispatching alert notifications to an
// Notifier is responsible for dispatching alert notifications to an
// alert manager service.
type Notifier struct {
queue model.Alerts
@ -61,7 +61,7 @@ type Notifier struct {
queueCapacity prometheus.Metric
}
// HandlerOptions are the configurable parameters of a Handler.
// Options are the configurable parameters of a Handler.
type Options struct {
AlertmanagerURL string
QueueCapacity int

3
promql/lex.go

@ -82,8 +82,7 @@ func (i itemType) isSetOperator() bool {
return false
}
// Constants for operator precedence in expressions.
//
// LowestPrec is a constant for operator precedence in expressions.
const LowestPrec = 0 // Non-operators.
// Precedence returns the operator precedence of the binary

21
promql/parse.go

@ -512,17 +512,16 @@ func (p *parser) balance(lhs Expr, op itemType, rhs Expr, vecMatching *VectorMat
VectorMatching: lhsBE.VectorMatching,
ReturnBool: lhsBE.ReturnBool,
}
} else {
if op.isComparisonOperator() && !returnBool && rhs.Type() == model.ValScalar && lhs.Type() == model.ValScalar {
p.errorf("comparisons between scalars must use BOOL modifier")
}
return &BinaryExpr{
Op: op,
LHS: lhs,
RHS: rhs,
VectorMatching: vecMatching,
ReturnBool: returnBool,
}
}
if op.isComparisonOperator() && !returnBool && rhs.Type() == model.ValScalar && lhs.Type() == model.ValScalar {
p.errorf("comparisons between scalars must use BOOL modifier")
}
return &BinaryExpr{
Op: op,
LHS: lhs,
RHS: rhs,
VectorMatching: vecMatching,
ReturnBool: returnBool,
}
}

2
storage/remote/graphite/client.go

@ -51,7 +51,7 @@ func pathFromMetric(m model.Metric, prefix string) string {
// We want to sort the labels.
labels := make(model.LabelNames, 0, len(m))
for l, _ := range m {
for l := range m {
labels = append(labels, l)
}
sort.Sort(labels)

Loading…
Cancel
Save