You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by "zhannngchen (via GitHub)" <gi...@apache.org> on 2023/06/27 11:14:13 UTC

[GitHub] [doris] zhannngchen opened a new pull request, #21254: [fix](partial-update) fix a coredump in commit_phase_update_delete_bitmap

zhannngchen opened a new pull request, #21254:
URL: https://github.com/apache/doris/pull/21254

   ## Proposed changes
   
   Issue Number: close #xxx
   
   ```
   SLF4J: Actual binding is of type [org.slf4j.impl.Reload4jLoggerFactory]
   F0627 14:53:59.750797 504076 segment_writer.cpp:333] Check failed: block->columns() > _tablet_schema->num_key_columns() && block->columns() < _tablet_schema->num_columns() block columns: 203, num key columns: 1,
   total schema columns: 203
   *** Check failure stack trace: ***
       @     0x55e7c081b83d  google::LogMessage::Fail()
       @     0x55e7c081dd79  google::LogMessage::SendToLog()
       @     0x55e7c081b3a6  google::LogMessage::Flush()
       @     0x55e7c081e3e9  google::LogMessageFatal::~LogMessageFatal()
       @     0x55e7b9a03478  doris::segment_v2::SegmentWriter::append_block_with_partial_content()
       @     0x55e7b9a0518e  doris::segment_v2::SegmentWriter::append_block()
       @     0x55e7b9932720  doris::BetaRowsetWriter::_do_add_block()
       @     0x55e7b992f7ed  doris::BetaRowsetWriter::_add_block()
       @     0x55e7b9933544  doris::BetaRowsetWriter::flush_single_memtable()
       @     0x55e7b9b4a810  doris::Tablet::calc_segment_delete_bitmap()
       @     0x55e7b9b4ca43  doris::Tablet::calc_delete_bitmap()
       @     0x55e7b9b4ee65  doris::Tablet::commit_phase_update_delete_bitmap()
       @     0x55e7b9be705d  doris::DeltaWriter::close_wait()
       @     0x55e7b9d3a95a  doris::TabletsChannel::_close_wait()
       @     0x55e7b9d3a423  doris::TabletsChannel::close()
       @     0x55e7b9c9076d  doris::LoadChannel::_handle_eos()
       @     0x55e7b9c8fb6e  doris::LoadChannel::add_batch()
       @     0x55e7b9c87765  doris::LoadChannelMgr::add_batch()
       @     0x55e7b9d80776  std::_Function_handler<>::_M_invoke()
       @     0x55e7b9324f62  doris::PriorityThreadPool::work_thread()
       @     0x55e7c3573890  execute_native_thread_routine
       @     0x7f1b176ed609  start_thread
       @     0x7f1b1797c133  clone
       @              (nil)  (unknown)
   *** Query id: 0-0 ***
   ```
   commit_phase_update_delete_bitmap can't use the original rowset writer directly, it should use a rowset writer with full schema.
   
   ## Further comments
   
   If this is a relatively large or complex change, kick off the discussion at [dev@doris.apache.org](mailto:dev@doris.apache.org) by explaining why you chose the solution you did and what alternatives you considered, etc...
   
   


-- 
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: commits-unsubscribe@doris.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


[GitHub] [doris] Yukang-Lian commented on pull request #21254: [fix](partial-update) fix a coredump in commit_phase_update_delete_bitmap

Posted by "Yukang-Lian (via GitHub)" <gi...@apache.org>.
Yukang-Lian commented on PR #21254:
URL: https://github.com/apache/doris/pull/21254#issuecomment-1609321087

   run buildall


-- 
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: commits-unsubscribe@doris.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


[GitHub] [doris] dataroaring merged pull request #21254: [fix](partial-update) fix a coredump in commit_phase_update_delete_bitmap

Posted by "dataroaring (via GitHub)" <gi...@apache.org>.
dataroaring merged PR #21254:
URL: https://github.com/apache/doris/pull/21254


-- 
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: commits-unsubscribe@doris.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


[GitHub] [doris] github-actions[bot] commented on pull request #21254: [fix](partial-update) fix a coredump in commit_phase_update_delete_bitmap

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #21254:
URL: https://github.com/apache/doris/pull/21254#issuecomment-1609312937

   PR approved by anyone and no changes requested.


-- 
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: commits-unsubscribe@doris.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


[GitHub] [doris] github-actions[bot] commented on pull request #21254: [fix](partial-update) fix a coredump in commit_phase_update_delete_bitmap

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #21254:
URL: https://github.com/apache/doris/pull/21254#issuecomment-1609302361

   clang-tidy review says "All clean, LGTM! :+1:"


-- 
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: commits-unsubscribe@doris.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org