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

Fixed a bug that needed to be released if data had null keys #26493

Merged
merged 2 commits into from Nov 8, 2023

Conversation

felixwluo
Copy link
Contributor

Proposed changes

Issue Number: close #xxx

Further comments

core

#0 0x000055a72acebb50 in doris::vectorized::IAggregateFunctionDataHelper<doris::vectorized::AggregateFunctionUniqExactData, doris::vectorized::AggregateFunctionUniq<long, doris::vectorized::AggregateFunctionUniqExactData > >::destroy(char*) const ()
#1 0x000055a72b3f26db in ZNSt8__detail9__variant17__gen_vtable_implINS0_12_Multi_arrayIPFNS0_21__deduce_visit_resultIvEEOZN5doris10vectorized15AggregationNode26_close_with_serialized_keyEvEUlOT_E_RSt7variantIJNS6_27AggregationMethodSerializedI9PHHashMapI9StringRefPc11DefaultHashISF_vELb0EEEENS6_26AggregationMethodOneNumberIh12FixedHashMapIhSG_28FixedHashMapImplicitZeroCellIhSG_16HashTableNoStateE28FixedHashTableCalculatedSizeISP_E9AllocatorILb1ELb1EEELb0EEENSL_ItSM_ItSG_SN_ItSG_SO_E24FixedHashTableStoredSizeISW_EST_ELb0EEENSL_IjSE_IjSG_9HashCRC32IjELb0EELb0EEENSL_ImSE_ImSG_S11_ImELb0EELb0EEENS6_30AggregationMethodStringNoCacheI13StringHashMapISG_ST_EEENSL_INS6_7UInt128ESE_IS1C_SG_S11_IS1C_ELb0EELb0EEENSL_IjSE_IjSG_14HashMixWrapperIjS12_ELb0EELb0EEENSL_ImSE_ImSG_S1G_ImS15_ELb0EELb0EEENSL_IS1C_SE_IS1C_SG_S1G_IS1C_S1D_ELb0EELb0EEENS6_37AggregationMethodSingleNullableColumnINSL_IhNS6_26AggregationDataWithNullKeyISU_EELb0EEEEENS1Q_INSL_ItNS1R_ISZ_EELb0EEEEENS1Q_INSL_IjNS1R_IS13_EELb0EEEEENS1Q_INSL_ImNS1R_IS16_EELb0EEEEENS1Q_INSL_IjNS1R_IS1I_EELb0EEEEENS1Q_INSL_ImNS1R_IS1L_EELb0EEEEENS1Q_INSL_IS1C_NS1R_IS1E_EELb0EEEEENS1Q_INSL_IS1C_NS1R_IS1O_EELb0EEEEENS1Q_INS18_INS1R_IS1A_EEEEEENS6_26AggregationMethodKeysFixedIS16_Lb0EEENS2J_IS16_Lb1EEENS2J_IS1E_Lb0EEENS2J_IS1E_Lb1EEENS2J_ISE_INS6_7UInt256ESG_S11_IS2O_ELb0EELb0EEENS2J_IS2Q_Lb1EEENS2J_IS1L_Lb0EEENS2J_IS1L_Lb1EEENS2J_IS1O_Lb0EEENS2J_IS1O_Lb1EEENS2J_ISE_IS2O_SG_S1G_IS2O_S2P_ELb0EELb0EEENS2J_IS2Y_Lb1EEEEEEJEEESt16integer_sequenceImJLm13EEEE14__visit_invokeESB_S32 ()
#2 0x000055a72b3f41b6 in doris::vectorized::AggregationNode::_close_with_serialized_key() ()
#3 0x000055a72b44957b in doris::vectorized::AggregationNode::close(doris::RuntimeState*) ()
#4 0x000055a72a1ea8f9 in doris::PlanFragmentExecutor::close() ()
#5 0x000055a72a1c92f5 in doris::FragmentExecState::execute() ()
#6 0x000055a72a1cc82e in doris::FragmentMgr::_exec_actual(std::shared_ptrdoris::FragmentExecState, std::function<void (doris::PlanFragmentExecutor*)>) ()
#7 0x000055a72a1ccd62 in std::_Function_handler<void (), doris::FragmentMgr::exec_plan_fragment(doris::TExecPlanFragmentParams const&, std::function<void (doris::PlanFragmentExecutor*)>)::{lambda()#1}>::_M_invoke(std::_Any_data const&) ()
#8 0x000055a72a486185 in doris::ThreadPool::dispatch_thread() ()
#9 0x000055a72a47b5df in doris::Thread::supervise_thread(void*) ()
#10 0x00007f1e77063ea5 in start_thread () from /lib64/libpthread.so.0
#11 0x00007f1e773769fd in clone () from /lib64/libc.so.6

@felixwluo
Copy link
Contributor Author

run buildall

@Gabriel39
Copy link
Contributor

Please add regression test cases

Copy link
Contributor

@yiguolei yiguolei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Nov 7, 2023
Copy link
Contributor

github-actions bot commented Nov 7, 2023

PR approved by at least one committer and no changes requested.

Copy link
Contributor

github-actions bot commented Nov 7, 2023

PR approved by anyone and no changes requested.

Copy link
Member

@xy720 xy720 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please check the clang format in pipeline

@felixwluo
Copy link
Contributor Author

Please check the clang format in pipeline

done

@felixwluo
Copy link
Contributor Author

run buildall

@lide-reed lide-reed self-requested a review November 7, 2023 12:26
@Gabriel39 Gabriel39 merged commit bce60e9 into apache:branch-1.2-lts Nov 8, 2023
15 of 18 checks passed
GoGoWen pushed a commit to GoGoWen/incubator-doris that referenced this pull request Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by one committer. area/vectorization dev/1.2.8-merged reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants