fix: Revert array_has_duplicates unit test due to Linux/Ubuntu test mismatch #12203
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
facebook-github-bot
added
the
CLA Signed
|
This pull request was exported from Phabricator. Differential Revision: D68845988 |
✅ Deploy Preview for meta-velox canceled.
|
bikramSingh91
approved these changes
Jan 29, 2025
peterenescu
changed the title
|
This pull request was exported from Phabricator. Differential Revision: D68845988 |
peterenescu
added a commit
to peterenescu/velox
that referenced
this pull request
Jan 29, 2025
…ismatch (facebookincubator#12203) Summary: Pull Request resolved: facebookincubator#12203 New unit test for array_has_duplicates fails on Linux but not Ubuntu. Undoing the addition temporarily while we investigate the issue in case it's a result of added Json functionality Reviewed By: bikramSingh91 Differential Revision: D68845988
peterenescu
force-pushed
the
export-D68845988
branch
from
55fe7bc to
c4a6319
Compare
|
This pull request was exported from Phabricator. Differential Revision: D68845988 |
1 similar comment
|
This pull request was exported from Phabricator. Differential Revision: D68845988 |
peterenescu
added a commit
to peterenescu/velox
that referenced
this pull request
Jan 29, 2025
…ismatch (facebookincubator#12203) Summary: Pull Request resolved: facebookincubator#12203 New unit test for array_has_duplicates fails on Linux but not Ubuntu. Undoing the addition temporarily while we investigate the issue in case it's a result of added Json functionality Reviewed By: bikramSingh91 Differential Revision: D68845988
peterenescu
force-pushed
the
export-D68845988
branch
from
c4a6319 to
9b48314
Compare
peterenescu
added a commit
to peterenescu/velox
that referenced
this pull request
Jan 29, 2025
…ismatch (facebookincubator#12203) Summary: New unit test for array_has_duplicates fails on Linux but not Ubuntu. Undoing the addition temporarily while we investigate the issue in case it's a result of added Json functionality Reviewed By: bikramSingh91 Differential Revision: D68845988
peterenescu
force-pushed
the
export-D68845988
branch
from
9b48314 to
9d8da09
Compare
|
This pull request was exported from Phabricator. Differential Revision: D68845988 |
1 similar comment
|
This pull request was exported from Phabricator. Differential Revision: D68845988 |
peterenescu
added a commit
to peterenescu/velox
that referenced
this pull request
Jan 29, 2025
…ismatch (facebookincubator#12203) Summary: Pull Request resolved: facebookincubator#12203 New unit test for array_has_duplicates fails on Linux but not Ubuntu. Undoing the addition temporarily while we investigate the issue in case it's a result of added Json functionality Reviewed By: bikramSingh91 Differential Revision: D68845988
peterenescu
force-pushed
the
export-D68845988
branch
from
9d8da09 to
4b9d150
Compare
…ismatch (facebookincubator#12203) Summary: New unit test for array_has_duplicates fails on Linux but not Ubuntu. Undoing the addition temporarily while we investigate the issue in case it's a result of added Json functionality Reviewed By: bikramSingh91 Differential Revision: D68845988
peterenescu
force-pushed
the
export-D68845988
branch
from
4b9d150 to
c410157
Compare
|
This pull request was exported from Phabricator. Differential Revision: D68845988 |
|
This pull request has been merged in 3fd46c7. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary: New unit test for array_has_duplicates fails on Linux but not Ubuntu. Undoing the addition temporarily while we investigate the issue in case it's a result of added Json functionality
Differential Revision: D68845988