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.

Sign up for GitHub

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

Jump to bottom

fix: add additional logging on loading fields.idxl files (#25309) #25319

Merged
merged 1 commit into from
Sep 12, 2024
Merged

fix: add additional logging on loading fields.idxl files (#25309) #25319

merged 1 commit into from
Sep 12, 2024

Conversation

davidby-influx
Copy link
Contributor

Log the path of the file being loaded, and when level=debug log progress fpr each set of field changes

closes #25289

(cherry picked from commit 5d8d112)

closes #25311

@davidby-influx
fix: add additional logging on loading fields.idxl files (#25309)
a677287 
Log the path of the file being loaded, and when level=debug
log progress fpr each set of field changes

closes #25289

(cherry picked from commit 5d8d112)

closes #25311
@davidby-influx davidby-influx self-assigned this Sep 12, 2024
@davidby-influx davidby-influx linked an issue Sep 12, 2024 that may be closed by this pull request
fields.idxl processing should print shard and file names [port to main-2.x] #25311
Closed
devanbenz
devanbenz approved these changes Sep 12, 2024
View reviewed changes
Copy link

@devanbenz devanbenz left a comment

Choose a reason for hiding this comment

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

Code looks similar -- LGTM!

@davidby-influx davidby-influx merged commit 96c97a7 into main-2.x Sep 12, 2024
25 checks passed
@davidby-influx davidby-influx deleted the DSB_field_logging_2 branch September 12, 2024 20:47
davidby-influx added a commit that referenced this pull request Sep 12, 2024
@davidby-influx
fix: add additional logging on loading fields.idxl files (#25309) (#2…
ea4540a 
…5319)

Log the path of the file being loaded, and when level=debug
log progress fpr each set of field changes

closes #25289

(cherry picked from commit 5d8d112)

closes #25311

(cherry picked from commit 96c97a7)

closes #25313
davidby-influx added a commit that referenced this pull request Sep 12, 2024
@davidby-influx
fix: add additional logging on loading fields.idxl files (#25309) (#2…
46086c8 
…5319) (#25324)

Log the path of the file being loaded, and when level=debug
log progress for each set of field changes

closes #25289

(cherry picked from commit 5d8d112)

closes #25311

(cherry picked from commit 96c97a7)

closes #25313
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@devanbenz devanbenz devanbenz approved these changes

@gwossum gwossum Awaiting requested review from gwossum

Assignees

@davidby-influx davidby-influx

Labels
area/logging area/tsm area/2.x OSS 2.0 related issues and PRs kind/enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

fields.idxl processing should print shard and file names [port to main-2.x]
2 participants
@davidby-influx @devanbenz