-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Make sure inputFlatNoNulls_ is initialized in ctor #11140
Conversation
✅ Deploy Preview for meta-velox canceled.
|
This pull request was exported from Phabricator. Differential Revision: D63673714 |
…#11140) Summary: Pull Request resolved: facebookincubator#11140 when `inputFlatNoNulls_` is not initialized, it could result in undefined behavior like one described in https://www.internalfb.com/diff/D63556283?dst_version_fbid=804905691596581&transaction_fbid=550949777494622 This diff initializes inputFlatNoNulls_ as false. Differential Revision: D63673714
38e066b
to
c8fd98d
Compare
This pull request was exported from Phabricator. Differential Revision: D63673714 |
…ncubator#11140) Summary: when `inputFlatNoNulls_` is not initialized, it could result in undefined behavior like one described in https://www.internalfb.com/diff/D63556283?dst_version_fbid=804905691596581&transaction_fbid=550949777494622 This diff initializes inputFlatNoNulls_ as false. Differential Revision: D63673714
c8fd98d
to
510858e
Compare
This pull request was exported from Phabricator. Differential Revision: D63673714 |
This pull request has been merged in e8db673. |
Conbench analyzed the 1 benchmark run on commit There were no benchmark performance regressions. 🎉 The full Conbench report has more details. |
Summary:
when
inputFlatNoNulls_
is not initialized, it could result in undefined behavior like one described in https://www.internalfb.com/diff/D63556283?dst_version_fbid=804905691596581&transaction_fbid=550949777494622This diff initializes inputFlatNoNulls_ as false.
Differential Revision: D63673714