Skip to content

[pull] master from ClickHouse:master#405

Merged
pull[bot] merged 4 commits intotraceon:masterfrom
ClickHouse:master
Nov 3, 2019
Merged

[pull] master from ClickHouse:master#405
pull[bot] merged 4 commits intotraceon:masterfrom
ClickHouse:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Nov 3, 2019

See Commits and Changes for more details.


Created by pull[bot]. Want to support this open source service? Please star it : )

azat and others added 4 commits November 3, 2019 21:44
Can be triggered using the following query:
  CREATE TABLE foo (key String, macro String MATERIALIZED __getScalar(key)) Engine=Null();

Trace:
    3. 0x00007ffff6d5d526 __assert_fail (libc.so.6)
    4. 0x00007ffff41fd931 boost::intrusive_ptr<DB::IColumn const>::operator*() const (libclickhouse_functionsd.so)
    5. 0x00007ffff41fcd64 COW<DB::IColumn>::IntrusivePtr<DB::IColumn const>::operator*() const & (libclickhouse_functionsd.so)
    6. 0x00007ffff4dc5944 DB::FunctionGetScalar::getReturnTypeImpl() const (libclickhouse_functionsd.so)

(Even though it is internal I guess it is better to fix it)

Refs: #7392
Cc: @amosbird
Check column for __getScalar() (avoids assertion)
@pull pull bot added the ⤵️ pull label Nov 3, 2019
@pull pull bot merged commit 8e2b1b3 into traceon:master Nov 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants