You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by "ege-st (via GitHub)" <gi...@apache.org> on 2023/07/17 16:17:16 UTC

[GitHub] [pinot] ege-st opened a new pull request, #11126: Revert jackson update

ege-st opened a new pull request, #11126:
URL: https://github.com/apache/pinot/pull/11126

   Revert the update to the Jackson library back to 2.12.7.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org


[GitHub] [pinot] snleee merged pull request #11126: Revert jackson update

Posted by "snleee (via GitHub)" <gi...@apache.org>.
snleee merged PR #11126:
URL: https://github.com/apache/pinot/pull/11126


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org


[GitHub] [pinot] mcvsubbu commented on pull request #11126: Revert jackson update

Posted by "mcvsubbu (via GitHub)" <gi...@apache.org>.
mcvsubbu commented on PR #11126:
URL: https://github.com/apache/pinot/pull/11126#issuecomment-1638880556

   Can we add some comments on why this is reverted?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org


[GitHub] [pinot] codecov-commenter commented on pull request #11126: Revert jackson update

Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #11126:
URL: https://github.com/apache/pinot/pull/11126#issuecomment-1638554367

   ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/11126?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
   > Merging [#11126](https://app.codecov.io/gh/apache/pinot/pull/11126?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) (6a7516e) into [master](https://app.codecov.io/gh/apache/pinot/commit/7422edfc1c65f0acd7cacd8227c7110cc05412c1?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) (7422edf) will **not change** coverage.
   > The diff coverage is `n/a`.
   
   ```diff
   @@           Coverage Diff           @@
   ##           master   #11126   +/-   ##
   =======================================
     Coverage    0.11%    0.11%           
   =======================================
     Files        2203     2203           
     Lines      118196   118196           
     Branches    17887    17887           
   =======================================
     Hits          137      137           
     Misses     118039   118039           
     Partials       20       20           
   ```
   
   | Flag | Coverage Δ | |
   |---|---|---|
   | integration1temurin11 | `0.00% <ø> (ø)` | |
   | integration1temurin17 | `0.00% <ø> (ø)` | |
   | integration1temurin20 | `0.00% <ø> (ø)` | |
   | integration2temurin11 | `?` | |
   | integration2temurin17 | `0.00% <ø> (ø)` | |
   | integration2temurin20 | `?` | |
   | unittests1temurin11 | `?` | |
   | unittests1temurin17 | `0.00% <ø> (ø)` | |
   | unittests1temurin20 | `0.00% <ø> (ø)` | |
   | unittests2temurin11 | `0.11% <ø> (?)` | |
   | unittests2temurin17 | `0.11% <ø> (ø)` | |
   | unittests2temurin20 | `0.11% <ø> (ø)` | |
   
   Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment) to find out more.
   
   
   :mega: We’re building smart automated test selection to slash your CI/CD build times. [Learn more](https://about.codecov.io/iterative-testing/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org