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

Send an explicit cancel request to the database instead of abandoning the connection #91

Closed
cornellb opened this issue Dec 3, 2019 · 1 comment
Assignees

Comments

@cornellb
Copy link

cornellb commented Dec 3, 2019

As I understand, the current way to deal with long running queries is to close the respective db connections (and create new ones to still maintain the configured pool size).

It should be more performant to send explicit cancel requests to the db. Some more info in the link below.
https://asktom.oracle.com/m/pls/asktom/asktom.search?tag=cancelling-long-running-queries

@ModEtchFill
Copy link
Contributor

We're working on merging a few more capabilities from our legacy oracleworker. This includes OCIBreak() canceling functionality.

@ModEtchFill ModEtchFill self-assigned this Jun 16, 2020
ModEtchFill added a commit that referenced this issue Jul 14, 2020
rasamala83 added a commit to rasamala83/hera that referenced this issue Aug 2, 2024
* occ configurations logging

* cal event success

* adding cal data

* added TODOs

* Remove log_occ_confogs.go

* Remove testing files

* source of configs - files

* whitelist format change

* code clean up

* code review changes-1

* CR fixes

* CR fixes

* Delete tests/unittest/config_logging/main_test.go

* clean up

* Merge branch 'occ-config-logging' of /Users/simmidisetty/Documents/GitHub/OpenSourceHera/src/github.com/paypal/hera with conflicts.

* test for config logging

* removing test changes

* tests for all cases

* test

* making minor changes for logging feature specific data

* changes for incorporate review comments

---------

Co-authored-by: satyakamala03 <128077872+satyakamala03@users.noreply.github.com>
Co-authored-by: simmidisetty <simmidisetty@paypal.com>
Co-authored-by: Rajesh S <samal.rajesh@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants