[PR] Fix NullArgumentException of getQuantile method in DescriptiveStatisticsHistogramStatistics [flink]

2023-12-12 Thread via GitHub


zhutong6688 opened a new pull request, #23915:
URL: https://github.com/apache/flink/pull/23915

   
   
   ## What is the purpose of the change
   
   Fix NullArgumentException of getQuantile method in 
DescriptiveStatisticsHistogramStatistics
   
   ## Brief change log
   
   This DescriptiveStatisticsHistogramStatistics class has added null 
initialization
   1、DescriptiveStatisticsHistogramStatistics.java
   
   ## Verifying this change
   
   https://github.com/apache/flink/assets/28818582/f498c368-0766-4cb1-adff-bb77abf5cda9";>
   
   During the verification process, the task was submitted to Yarn and run in 
Yarn cluster mode. After multiple stops and restarts, it was found that the 
NullArgumentException exception had disappeared. This fully verified the 
correctness of the modification, improved the stability of program operation, 
and also improved the accuracy of displaying statistical information in the web 
UI interface of the Flink task (as shown in the above figure).
   The highlighted log section in the above figure is the log I printed to 
assist in troubleshooting, which can be ignored.
   
   ## Does this pull request potentially affect one of the following parts:
   
 - Dependencies (does it add or upgrade a dependency): (no)
 - The public API, i.e., is any changed class annotated with 
`@Public(Evolving)`: (no)
 - The serializers: (no)
 - The runtime per-record code paths (performance sensitive): (no)
 - Anything that affects deployment or recovery: JobManager (and its 
components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes)
 - The S3 file system connector: (no)
   
   ## Documentation
   
 - Does this pull request introduce a new feature? (no)
 - If yes, how is the feature documented? (no)


-- 
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: issues-unsubscr...@flink.apache.org

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



[PR] Fix NullArgumentException of getQuantile method in DescriptiveStatisticsHistogramStatistics [flink]

2023-12-14 Thread via GitHub


zhutong6688 opened a new pull request, #23931:
URL: https://github.com/apache/flink/pull/23931

   
   
   ## What is the purpose of the change
   
   Fix NullArgumentException of getQuantile method in 
DescriptiveStatisticsHistogramStatistics
   
   ## Brief change log
   
   This DescriptiveStatisticsHistogramStatistics class has added null 
initialization
   1、DescriptiveStatisticsHistogramStatistics.java
   
   ## Verifying this change
   
   https://github.com/apache/flink/assets/28818582/f498c368-0766-4cb1-adff-bb77abf5cda9";>
   
   During the verification process, the task was submitted to Yarn and run in 
Yarn cluster mode. After multiple stops and restarts, it was found that the 
NullArgumentException exception had disappeared. This fully verified the 
correctness of the modification, improved the stability of program operation, 
and also improved the accuracy of displaying statistical information in the web 
UI interface of the Flink task (as shown in the above figure).
   The highlighted log section in the above figure is the log I printed to 
assist in troubleshooting, which can be ignored.
   
   ## Does this pull request potentially affect one of the following parts:
   
 - Dependencies (does it add or upgrade a dependency): (no)
 - The public API, i.e., is any changed class annotated with 
`@Public(Evolving)`: (no)
 - The serializers: (no)
 - The runtime per-record code paths (performance sensitive): (no)
 - Anything that affects deployment or recovery: JobManager (and its 
components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes)
 - The S3 file system connector: (no)
   
   ## Documentation
   
 - Does this pull request introduce a new feature? (no)
 - If yes, how is the feature documented? (no)


-- 
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: issues-unsubscr...@flink.apache.org

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



[PR] Fix NullArgumentException of getQuantile method in DescriptiveStatisticsHistogramStatistics [flink]

2023-12-18 Thread via GitHub


JingGe opened a new pull request, #23947:
URL: https://github.com/apache/flink/pull/23947

   BP of #23931 with no change
   


-- 
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: issues-unsubscr...@flink.apache.org

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



Re: [PR] Fix NullArgumentException of getQuantile method in DescriptiveStatisticsHistogramStatistics [flink]

2023-12-12 Thread via GitHub


zhutong6688 commented on PR #23915:
URL: https://github.com/apache/flink/pull/23915#issuecomment-1852391071

   issue link is https://issues.apache.org/jira/browse/FLINK-33588


-- 
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: issues-unsubscr...@flink.apache.org

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



Re: [PR] Fix NullArgumentException of getQuantile method in DescriptiveStatisticsHistogramStatistics [flink]

2023-12-12 Thread via GitHub


zhutong6688 commented on PR #23915:
URL: https://github.com/apache/flink/pull/23915#issuecomment-1852394178

   old PR is https://github.com/apache/flink/pull/23749, #23749, this PR is 
new, but the content is the same


-- 
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: issues-unsubscr...@flink.apache.org

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



Re: [PR] Fix NullArgumentException of getQuantile method in DescriptiveStatisticsHistogramStatistics [flink]

2023-12-14 Thread via GitHub


flinkbot commented on PR #23931:
URL: https://github.com/apache/flink/pull/23931#issuecomment-1856171367

   
   ## CI report:
   
   * ee5a7de3a7921a54c8561cb20ab7c56f2e6c7de7 UNKNOWN
   
   
   Bot commands
 The @flinkbot bot supports the following commands:
   
- `@flinkbot run azure` re-run the last Azure build
   


-- 
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: issues-unsubscr...@flink.apache.org

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



Re: [PR] Fix NullArgumentException of getQuantile method in DescriptiveStatisticsHistogramStatistics [flink]

2023-12-18 Thread via GitHub


JingGe commented on PR #23947:
URL: https://github.com/apache/flink/pull/23947#issuecomment-1860197970

   @flinkbot run azure


-- 
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: issues-unsubscr...@flink.apache.org

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



Re: [PR] Fix NullArgumentException of getQuantile method in DescriptiveStatisticsHistogramStatistics [flink]

2023-12-18 Thread via GitHub


flinkbot commented on PR #23947:
URL: https://github.com/apache/flink/pull/23947#issuecomment-1860198985

   
   ## CI report:
   
   * 28314d4fed2721dbabac426b41a416c79df772f0 UNKNOWN
   
   
   Bot commands
 The @flinkbot bot supports the following commands:
   
- `@flinkbot run azure` re-run the last Azure build
   


-- 
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: issues-unsubscr...@flink.apache.org

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



Re: [PR] Fix NullArgumentException of getQuantile method in DescriptiveStatisticsHistogramStatistics [flink]

2023-12-18 Thread via GitHub


JingGe commented on PR #23947:
URL: https://github.com/apache/flink/pull/23947#issuecomment-1860270081

   @flinkbot run azure


-- 
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: issues-unsubscr...@flink.apache.org

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