Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[UI - Buckets] Sort by Retention Policy seems to be using Alpha Numeric sort and not retentionRules.everySeconds #14923

Closed
karel-rehor opened this issue Sep 4, 2019 · 1 comment · Fixed by #15728
Assignees
Labels
area/ui kind/bug ui/design-polish Something in the UI needs a small tweak (not a whole redesign)

Comments

@karel-rehor
Copy link
Contributor

Steps to reproduce:
List the minimal actions needed to reproduce the behavior.

  1. In the UI > LoadData > Buckets list create a few new buckets
  2. Define for each bucket various Retention Policy lengths
  3. Sort the list of buckets by Retention Policy

Sort appears to occur by Alphanumeric value not by length of the retention policy.

Expected behavior:
Looking at the results from the REST endpoint it appears that Retention Policies are now reported as an array of object {type: "expire", everySeconds: long}. I do not see a name property associated with Retention Policies, so I would expect the Retention Policy sort to work by the length of the policy. Without a name, an alphanumeric sort makes little sense.

Expected sort from current example:

  1. 1 Hours = 3600s
  2. 12 Hours = 43200s
  3. 1 Days = 86400s
  4. 1 Days 12 Hours = 129600s
  5. 3 Days = 259200s
  6. 28 Days = 2419200s

Actual behavior:
It appears the sort is alphanumeric: e.g.

Actual sort of current example:

  1. 1 Days
  2. 1 Days 12 Hours
  3. 1 Hours
  4. 12 Hours
  5. 28 Days
  6. 3 Days

Screenshot
BucketsSortByRetention01

Environment info:

  • System info: Run uname -srm and copy the output here
    Linux 4.4.0-161-generic x86_64
  • InfluxDB version: Run influxd version and copy the output here
    This is from the nightly build docker image.
    From the docker log:
ts=2019-09-04T08:18:21.843812Z lvl=info msg="Welcome to InfluxDB" log_id=0HfqVC4l000 version=nightly commit=20f717dcf build_date=2019-09-04T07:22:16Z
  • Other relevant environment details: Container runtime, disk info, etc

Config:
Started with /debug/flush enabled

--e2e-testing=true
@karel-rehor
Copy link
Contributor Author

Related Issues
#15397
#15301
#15247
#13950

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ui kind/bug ui/design-polish Something in the UI needs a small tweak (not a whole redesign)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants