Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable approx_distinct result verifier in window fuzzer test #9407

Closed
wants to merge 1 commit into from

Conversation

kagamiori
Copy link
Contributor

Summary: The approx_distinct result verifier for window fuzzer is giving false positive signals. It need to be fixed as described in #9347. Disable this result verifier until it is fixed.

Differential Revision: D55879765

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 8, 2024
…okincubator#9407)

Summary:

The approx_distinct result verifier for window fuzzer is giving false positive signals. It need to be fixed as described in facebookincubator#9347. Disable this result verifier until it is fixed.

Differential Revision: D55879765
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D55879765

Copy link

netlify bot commented Apr 8, 2024

Deploy Preview for meta-velox canceled.

Name Link
🔨 Latest commit 4401715
🔍 Latest deploy log https://app.netlify.com/sites/meta-velox/deploys/66142cb0a8c4e800078d2819

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D55879765

Copy link

netlify bot commented Apr 8, 2024

Deploy Preview for meta-velox canceled.

Name Link
🔨 Latest commit 9da1e70
🔍 Latest deploy log https://app.netlify.com/sites/meta-velox/deploys/66142cc3825047000731ec9e

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in fa177d3.

Copy link

Conbench analyzed the 1 benchmark run on commit fa177d33.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details.

Joe-Abraham pushed a commit to Joe-Abraham/velox that referenced this pull request Jun 7, 2024
…okincubator#9407)

Summary:
Pull Request resolved: facebookincubator#9407

The approx_distinct result verifier for window fuzzer is giving false positive signals. It need to be fixed as described in facebookincubator#9347. Disable this result verifier until it is fixed.

Reviewed By: kgpai

Differential Revision: D55879765

fbshipit-source-id: 23d3c30cc4660ccb111ef641ed22ef111f79e6a4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants