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

Fix JDBC Log query order. #11544

Merged
merged 1 commit into from
Nov 15, 2023

Conversation

weixiang1862
Copy link
Member

Fix JDBC Log query order

  • Add a unit test to verify that the fix works.
  • Explain briefly why the bug exists and how to fix it.
  • If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
  • Update the CHANGES log.

Logs query is order by timestamp, ASC means comparing(Log::getTimestamp) but not comparing(Log::getTimestamp).reversed();

@wu-sheng wu-sheng added this to the 9.7.0 milestone Nov 15, 2023
@wu-sheng wu-sheng added bug Something isn't working and you are sure it's a bug! backend OAP backend related. query OAP query protocol related logging Log collecting and analysis labels Nov 15, 2023
@wu-sheng wu-sheng merged commit eaac1c3 into apache:master Nov 15, 2023
166 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend OAP backend related. bug Something isn't working and you are sure it's a bug! logging Log collecting and analysis query OAP query protocol related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants