Skip to content

SOLR-16535 Remove jaegertracer-configurator module in 10.0#1272

Merged
janhoy merged 4 commits intoapache:mainfrom
janhoy:SOLR-16535-Remove_jaegertracer-configurator_module
Jan 6, 2023
Merged

SOLR-16535 Remove jaegertracer-configurator module in 10.0#1272
janhoy merged 4 commits intoapache:mainfrom
janhoy:SOLR-16535-Remove_jaegertracer-configurator_module

Conversation

@janhoy
Copy link
Contributor

@janhoy janhoy commented Jan 4, 2023

@janhoy janhoy requested review from dsmiley, epugh and risdenk January 4, 2023 21:58
@janhoy janhoy changed the title Solr 16535 Remove jaegertracer-configurator module in 10.0 SOLR-16535 Remove jaegertracer-configurator module in 10.0 Jan 4, 2023
Copy link
Contributor

@epugh epugh left a comment

Choose a reason for hiding this comment

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

LGTM.

@janhoy janhoy force-pushed the SOLR-16535-Remove_jaegertracer-configurator_module branch from 546290e to 24589cc Compare January 5, 2023 23:52
org.apache.sis.core:sis-utility:1.2 (4 constraints: 0b35ce5b)
org.apache.sis.storage:sis-netcdf:1.2 (1 constraints: fc0b54e9)
org.apache.sis.storage:sis-storage:1.2 (1 constraints: df0de432)
org.apache.thrift:libthrift:0.15.0 (1 constraints: 900d0334)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yey, finally this is gone :)

@janhoy janhoy requested a review from epugh January 6, 2023 00:08
@janhoy
Copy link
Contributor Author

janhoy commented Jan 6, 2023

This is now ready after some final edits, and addition of refGuide major-changes note for 10.0.
+21 lines and −1,084 lines. Love it :)

I'll merge whenever I get a +1 or if silent, on Tuesday 10th

@janhoy janhoy merged commit ae8d3e5 into apache:main Jan 6, 2023
@janhoy janhoy deleted the SOLR-16535-Remove_jaegertracer-configurator_module branch January 6, 2023 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants