[fix][fn] ack messages for window function when its result is null

[jiangpengcheng]

Fixes #23617

Main Issue: #xyz

PIP: #xyz

Motivation

The window function doesn't handle ack correctly

Modifications

Verifying this change

• Make sure that the change passes the CI checks.

• This change is already covered by existing tests, such as (please describe tests).

  • org.apache.pulsar.tests.integration.functions.java.testTumblingCountWindowTest
  • org.apache.pulsar.tests.integration.functions.java.testSlidingCountWindowTest

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

• Dependencies (add or upgrade a dependency)
• The public API
• The schema
• The default values of configurations
• The threading model
• The binary protocol
• The REST endpoints
• The admin CLI options
• The metrics
• Anything that affects deployment

Documentation

• doc
• doc-required
• doc-not-needed
• doc-complete

Matching PR in forked repository

PR in forked repository: jiangpengcheng#34

[shibd]

/pulsarbot rerun-failure-checks

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Project coverage is 74.39%. Comparing base (bbc6224) to head (a27e08d).
Report is 743 commits behind head on master.

Files with missing lines	Patch %	Lines
...ar/functions/windowing/WindowFunctionExecutor.java	0.00%	3 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #23618      +/-   ##
============================================
+ Coverage     73.57%   74.39%   +0.82%     
- Complexity    32624    34469    +1845     
============================================
  Files          1877     1944      +67     
  Lines        139502   147134    +7632     
  Branches      15299    16226     +927     
============================================
+ Hits         102638   109467    +6829     
- Misses        28908    29233     +325     
- Partials       7956     8434     +478     

Flag	Coverage Δ
inttests	27.51% <0.00%> (+2.92%)	⬆️
systests	24.33% <0.00%> (+0.01%)	⬆️
unittests	73.78% <0.00%> (+0.94%)	⬆️

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

Files with missing lines	Coverage Δ
...ar/functions/windowing/WindowFunctionExecutor.java	66.43% <0.00%> (-1.92%)	⬇️

... and 663 files with indirect coverage changes

---- 🚨 Try these New Features:

• Flaky Tests Detection - Detect and resolve failed and flaky tests
• JS Bundle Analysis - Avoid shipping oversized bundles

[jiangpengcheng]

/pulsarbot rerun-failure-checks