Re: [PR] Streaming brokers [pinot]

via GitHub Wed, 04 Feb 2026 10:52:12 -0800


codecov-commenter commented on PR #17613:
URL: https://github.com/apache/pinot/pull/17613#issuecomment-3849171480


   ### :x: 3 Tests Failed:
   | Tests completed | Failed | Passed | Skipped |
   |---|---|---|---|
   | 6412 | 3 | 6409 | 0 |
   <details><summary>View the top 3 failed test(s) by shortest run 
time</summary>
   
   > <pre><code 
class="language-python">org.apache.pinot.broker.api.resources.PinotClientRequestTest::testPinotQueryComparisonApiDifferentQuery</code></pre>
   > <details><summary>Stack Traces | 0.086s run time</summary>
   > 
   > > <pre><code class="language-python">
   > > Wanted but not invoked:
   > > _requestHandler.handleRequest(
   > >     &lt;Capturing argument: JsonNode&gt;,
   > >     &lt;any&gt;,
   > >     &lt;any&gt;,
   > >     &lt;any&gt;,
   > >     &lt;any&gt;
   > > );
   > > -&gt; at 
org.apache.pinot.broker.api.resources.PinotClientRequestTest.testPinotQueryComparisonApiDifferentQuery(PinotClientRequestTest.java:165)
   > > 
   > > However, there were exactly 2 interactions with this mock:
   > > _requestHandler.handleStreamingRequest(
   > >     {&quot;sqlV1&quot;:&quot;SELECT v1 FROM 
mytable&quot;,&quot;sqlV2&quot;:&quot;SELECT v2 FROM 
mytable&quot;,&quot;sql&quot;:&quot;SELECT v1 FROM mytable&quot;},
   > >     org.apache.pinot.sql.parsers.SqlNodeAndOptions@42c3b02e,
   > >     org.apache.pinot.broker.api.HttpRequesterIdentity@51039ec9,
   > >     org.apache.pinot.spi.trace.DefaultRequestContext@16575bfe,
   > >     null
   > > );
   > > -&gt; at 
org.apache.pinot.broker.api.resources.PinotClientRequest.executeSqlQuery(PinotClientRequest.java:614)
   > > 
   > > _requestHandler.handleStreamingRequest(
   > >     {&quot;sqlV1&quot;:&quot;SELECT v1 FROM 
mytable&quot;,&quot;sqlV2&quot;:&quot;SELECT v2 FROM 
mytable&quot;,&quot;sql&quot;:&quot;SELECT v2 FROM mytable&quot;},
   > >     org.apache.pinot.sql.parsers.SqlNodeAndOptions@60003305,
   > >     org.apache.pinot.broker.api.HttpRequesterIdentity@3f1dadeb,
   > >     org.apache.pinot.spi.trace.DefaultRequestContext@72259917,
   > >     null
   > > );
   > > -&gt; at 
org.apache.pinot.broker.api.resources.PinotClientRequest.executeSqlQuery(PinotClientRequest.java:614)
   > > 
   > > </code></pre>
   > 
   > </details>
   
   
   > <pre><code 
class="language-python">org.apache.pinot.broker.api.resources.PinotClientRequestTest::testGetPinotQueryResponse</code></pre>
   > <details><summary>Stack Traces | 0.123s run time</summary>
   > 
   > > <pre><code class="language-python">expected [200] but found 
[404]</code></pre>
   > 
   > </details>
   
   
   > <pre><code 
class="language-python">org.apache.pinot.broker.api.resources.PinotClientRequestTest::testPinotQueryComparisonApiSameQuery</code></pre>
   > <details><summary>Stack Traces | 1.01s run time</summary>
   > 
   > > <pre><code class="language-python">
   > > Wanted but not invoked:
   > > _requestHandler.handleRequest(
   > >     &lt;Capturing argument: JsonNode&gt;,
   > >     &lt;Capturing argument: SqlNodeAndOptions&gt;,
   > >     &lt;any&gt;,
   > >     &lt;any&gt;,
   > >     &lt;any&gt;
   > > );
   > > -&gt; at 
org.apache.pinot.broker.api.resources.PinotClientRequestTest.testPinotQueryComparisonApiSameQuery(PinotClientRequestTest.java:139)
   > > 
   > > However, there were exactly 2 interactions with this mock:
   > > _requestHandler.handleStreamingRequest(
   > >     {&quot;sql&quot;:&quot;SELECT * FROM mytable&quot;},
   > >     org.apache.pinot.sql.parsers.SqlNodeAndOptions@77c4aeb6,
   > >     org.apache.pinot.broker.api.HttpRequesterIdentity@54455c8d,
   > >     org.apache.pinot.spi.trace.DefaultRequestContext@80347b2,
   > >     null
   > > );
   > > -&gt; at 
org.apache.pinot.broker.api.resources.PinotClientRequest.executeSqlQuery(PinotClientRequest.java:614)
   > > 
   > > _requestHandler.handleStreamingRequest(
   > >     {&quot;sql&quot;:&quot;SELECT * FROM mytable&quot;},
   > >     org.apache.pinot.sql.parsers.SqlNodeAndOptions@eb65fc8,
   > >     org.apache.pinot.broker.api.HttpRequesterIdentity@5cd5e1b6,
   > >     org.apache.pinot.spi.trace.DefaultRequestContext@220a7f95,
   > >     null
   > > );
   > > -&gt; at 
org.apache.pinot.broker.api.resources.PinotClientRequest.executeSqlQuery(PinotClientRequest.java:614)
   > > 
   > > </code></pre>
   > 
   > </details>
   
   </details>
   
   To view more test analytics, go to the [Test Analytics 
Dashboard](https://app.codecov.io/gh/apache/pinot/tests/gortiz%3Alazy-brokers)
   <sub>📋 Got 3 mins? [Take this short 
survey](https://forms.gle/22i53Qa1CySZjA6c7) to help us improve Test 
Analytics.</sub>


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Re: [PR] Streaming brokers [pinot]

Reply via email to