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

Sort topic data OLDEST / NEWEST #460

Closed
PLarboulette opened this issue Oct 20, 2020 · 3 comments · Fixed by #511
Closed

Sort topic data OLDEST / NEWEST #460

PLarboulette opened this issue Oct 20, 2020 · 3 comments · Fixed by #511
Labels
bug Something isn't working ui Need an ui update

Comments

@PLarboulette
Copy link
Contributor

Hello,

I think the
topic-data: sort: NEWEST
is broken.

Even if the parameter is set to NEWEST, topics are sorted by the default value, OLDEST.

@tchiotludo
Copy link
Owner

Seems that you don't respect yaml indent.
Can you paste the whole file please ?

@PLarboulette
Copy link
Contributor Author

PLarboulette commented Oct 20, 2020

akhq:
  server:
    access-log:
      enabled: true
      name: org.akhq.log.access
      format: "[Date: {}] [Duration: {} ms] [Url: {} {} {}] [Status: {}] [Ip: {}]
        [Length: {}] [Port: {}]"
  clients-defaults:
    consumer:
      properties:
        isolation.level: read_committed
  connections:
    kafka-cluster:
      properties:
        bootstrap.servers: "${KAFKA_BOOTSTRAP_SERVERS}"
      schema-registry:
        url: "${SCHEMA_REGISTRY_URL}"
      connect:
        - name: "${KAFKA_CONNECT_NAME}"
          url: "${KAFKA_CONNECT_URL}"
  pagination:
    page-size: 25
    threads: 16
  topic-data:
    sort: NEWEST
    size: 50
    poll-timeout: 1000

topic-data and pagination are in the same levels as connections or server. When I look the example file, it seems to be the same thing.

@tchiotludo
Copy link
Owner

Ok got it :)
This option is not transmitted to client so It can't work :)

Good catch 👍

@tchiotludo tchiotludo added bug Something isn't working ui Need an ui update labels Oct 20, 2020
@tchiotludo tchiotludo assigned ghost Oct 20, 2020
@ghost ghost linked a pull request Nov 16, 2020 that will close this issue
tchiotludo pushed a commit that referenced this issue Nov 20, 2020
* global options 
* cluster options
* all of these configurable directly on ui per cluster

close #460 
close #506
close #464
relate #254
relate #466
relate #500
@tchiotludo tchiotludo moved this to Done in Backlog Jun 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ui Need an ui update
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants