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

chore: use logforth to replace fern #16249

Merged
merged 6 commits into from
Sep 12, 2024
Merged

Conversation

andylokandy
Copy link
Collaborator

@andylokandy andylokandy commented Aug 14, 2024

I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/

Summary

Tests

  • Unit Test
  • Logic Test
  • Benchmark Test
  • No Test - Explain why

Type of change

  • Bug Fix (non-breaking change which fixes an issue)
  • New Feature (non-breaking change which adds functionality)
  • Breaking Change (fix or feature that could cause existing functionality not to work as expected)
  • Documentation Update
  • Refactoring
  • Performance Improvement
  • Other (please describe):

This change is Reviewable

@github-actions github-actions bot added the pr-chore this PR only has small changes that no need to record, like coding styles. label Aug 14, 2024
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@sundy-li
Copy link
Member

There are extra error messages:

+ (while in processor thread 0)

Need fix the ci tests.

@andylokandy andylokandy marked this pull request as draft September 3, 2024 10:39
@andylokandy andylokandy marked this pull request as ready for review September 11, 2024 17:30
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. pr-refactor this PR changes the code base without new features or bugfix labels Sep 11, 2024
@sundy-li sundy-li added this pull request to the merge queue Sep 12, 2024
Merged via the queue into databendlabs:main with commit a620190 Sep 12, 2024
71 checks passed
serde_json::to_string(&fields).unwrap_or_default(),
));
}
Some(query_id) => {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

query_id missing for the new system logging cc @andylokandy

@BohuTANG
Copy link
Member

This PR introduces a breaking change to the design. Cloud query history and system logs are all broken now.

andylokandy added a commit to andylokandy/databend that referenced this pull request Sep 19, 2024
BohuTANG added a commit that referenced this pull request Sep 19, 2024
* fix: revert log format breakage from #16249

* fix

---------

Co-authored-by: Bohu <[email protected]>
@andylokandy andylokandy deleted the dev3 branch September 19, 2024 08:42
dantengsky pushed a commit to dantengsky/fuse-query that referenced this pull request Oct 28, 2024
dantengsky added a commit that referenced this pull request Oct 28, 2024
* fix: revert log format breakage from #16249

* fix

---------

Co-authored-by: andylokandy <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-chore this PR only has small changes that no need to record, like coding styles. pr-refactor this PR changes the code base without new features or bugfix size:XL This PR changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants