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

Fix use_environment_credentials for s3 table function #41970

Merged
merged 1 commit into from
Oct 1, 2022

Conversation

kssenii
Copy link
Member

@kssenii kssenii commented Sep 30, 2022

Changelog category (leave one):

  • Bug Fix (user-visible misbehavior in official stable or prestable release)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Fix use_environment_credentials for s3 table function

@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Sep 30, 2022
@kssenii
Copy link
Member Author

kssenii commented Oct 1, 2022

Integration tests

#41933

Stateless tests (aarch64) — fail: 1

02267_file_globs_schema_inference

Stateless tests (debug) [1/3] — fail: 1

02122_join_group_by_timeout

Stress test (msan) — Sanitizer assert (in stderr.log)

==1472==WARNING: MemorySanitizer: use-of-uninitialized-value
    #0 0x4a487819 in llvm::iplist_impl<llvm::simple_ilist<llvm::MachineInstr, llvm::ilist_sentinel_tracking<true>>, llvm::ilist_traits<llvm::MachineInstr>>::~iplist_impl() MachineBasicBlock.cpp
    #1 0x4a487c3f in llvm::MachineBasicBlock::~MachineBasicBlock() (/usr/bin/clickhouse+0x4a487c3f) (BuildId: 4690abd78410f46accec3c9ce65e6c7684c343c5)
    #2 0x4a5ffc3f in llvm::ilist_alloc_traits<llvm::MachineBasicBlock>::deleteNode(llvm::MachineBasicBlock*) (/usr/bin/clickhouse+0x4a5ffc3f) (BuildId: 4690abd78410f46accec3c9ce65e6c7684c343c5)
    #3 0x4a4b39d6 in llvm::MachineBasicBlock::eraseFromParent() (/usr/bin/clickhouse+0x4a4b39d6) (BuildId: 4690abd78410f46accec3c9ce65e6c7684c343c5)
    #4 0x4adb54c7 in llvm::TailDuplicator::removeDeadBlock(llvm::MachineBasicBlock*, llvm::function_ref<void (llvm::MachineBasicBlock*)>*) (/usr/bin/clickhouse+0x4adb54c7) (BuildId: 4690abd78410f46accec3c9ce65e6c7684c343c5)

Stress test (debug) — Hung check failed

not related

@kssenii kssenii merged commit 59596e9 into ClickHouse:master Oct 1, 2022
@robot-ch-test-poll robot-ch-test-poll added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Oct 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-bugfix Pull request with bugfix, not backported by default
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants