Skip to content

Conversation

@walterddr
Copy link
Contributor

follow up on #9702. I misunderstood the upgrade order to be server-->broker->controller; with this change we are good for controller->broker->server

@walterddr walterddr changed the title [hotfix] use legacy format before releasing 0.13 [hotfix] use legacy case-when format before releasing 0.13 Feb 16, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #10291 (2e7522d) into master (b19f3fe) will increase coverage by 56.70%.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##             master   #10291       +/-   ##
=============================================
+ Coverage     13.71%   70.42%   +56.70%     
- Complexity      182     5117     +4935     
=============================================
  Files          1961     2016       +55     
  Lines        106866   109636     +2770     
  Branches      16326    16681      +355     
=============================================
+ Hits          14661    77214    +62553     
+ Misses        91046    26996    -64050     
- Partials       1159     5426     +4267     
Flag Coverage Δ
integration1 24.63% <100.00%> (?)
integration2 24.38% <100.00%> (?)
unittests1 67.68% <100.00%> (?)
unittests2 13.70% <0.00%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...org/apache/pinot/sql/parsers/CalciteSqlParser.java 85.05% <100.00%> (+85.05%) ⬆️
...troller/helix/core/retention/RetentionManager.java 79.03% <0.00%> (ø)
...pache/pinot/plugin/metrics/yammer/YammerGauge.java 83.33% <0.00%> (ø)
.../metrics/yammer/YammerMetricsRegistryListener.java 0.00% <0.00%> (ø)
...pache/pinot/plugin/metrics/yammer/YammerMeter.java 55.55% <0.00%> (ø)
...ver/api/resources/ControllerJobStatusResource.java 60.00% <0.00%> (ø)
...inot/server/api/resources/ServerResourceUtils.java 50.00% <0.00%> (ø)
.../apache/pinot/server/worker/WorkerQueryServer.java 70.00% <0.00%> (ø)
.../helix/FreshnessBasedConsumptionStatusChecker.java 0.00% <0.00%> (ø)
...pinot/plugin/metrics/yammer/YammerJmxReporter.java 100.00% <0.00%> (ø)
... and 1587 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@mayankshriv mayankshriv left a comment

Choose a reason for hiding this comment

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

@walterddr are we reverting the change, what's the plan on zero downtime upgrade for next 0.13? Or will that not have an issue?

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.

4 participants