-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[doc][2.25.1] Add flags for Advisory locks #25659
base: master
Are you sure you want to change the base?
Changes from all commits
b9f47b5
4033800
0ea5041
40461b1
f0fdaeb
9183cb0
a7a8010
e069e8e
74f6508
c8eee4a
b7bc7cd
5766040
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -210,9 +210,9 @@ ERROR: partman is not a valid partitioning type for pg_partman | |
|
||
The pg_partman `create_parent()` function requires an access exclusive lock on the parent table to create new child partitions. Currently, access exclusive locks are not supported in YugabyteDB, and are disabled in this function. | ||
|
||
### ADVISORY LOCKS | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What if you enable pg_partman but don't set the advisory lock flags for the cluster There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Then the advisory locks calls will fail with the message to enable the preview feature. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Looks like there is dev work to get pg_partman to use advisory_locks. Will undo this change for now. |
||
### Advisory locks | ||
|
||
Advisory locks, used in some pg_partman functions to create, drop/delete partitioned tables, are not supported in YugabyteDB. Attempts to acquire these locks are disabled. | ||
Advisory locks, used in some pg_partman functions to create, drop/delete partitioned tables, are not yet supported in YugabyteDB. Attempts to acquire these locks are disabled. | ||
|
||
### Background worker process | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -46,7 +46,6 @@ Review limitations and implement suggested workarounds to successfully migrate d | |
- [XID functions is not supported](#xid-functions-is-not-supported) | ||
- [REFERENCING clause for triggers](#referencing-clause-for-triggers) | ||
- [BEFORE ROW triggers on partitioned tables](#before-row-triggers-on-partitioned-tables) | ||
- [Advisory locks is not yet implemented](#advisory-locks-is-not-yet-implemented) | ||
- [System columns is not yet supported](#system-columns-is-not-yet-supported) | ||
- [XML functions is not yet supported](#xml-functions-is-not-yet-supported) | ||
- [Large Objects and its functions are currently not supported](#large-objects-and-its-functions-are-currently-not-supported) | ||
|
@@ -1240,25 +1239,6 @@ EXECUTE FUNCTION check_and_modify_val(); | |
|
||
--- | ||
|
||
### Advisory locks is not yet implemented | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Has Voyager been updated to know that advisory locks are supported? And it is only so in 2.25.1, what if they migrate to a diffferent version? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @ddhodge , Voyager changes are tracked by https://yugabyte.atlassian.net/browse/DB-13780. This update to the documentation is independant. Do you want the document changes to be done only after the Voyager changes land on master, so as to be consistent? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Voyager uses the link to this section in assessment reports and we have a different workflow to mark a feature as fixed in certain versions, we have to do this separately once Voyager also supports the new YB version where Advisory lock is implemented. We generally don't remove the section in this docs, we just mention the YB version where the feature is fixed. |
||
|
||
**GitHub**: [Issue #3642](https://github.com/yugabyte/yugabyte-db/issues/3642) | ||
|
||
**Description**: YugabyteDB does not support PostgreSQL advisory locks (for example, pg_advisory_lock, pg_try_advisory_lock). Any attempt to use advisory locks will result in a "function-not-implemented" error as per the following example: | ||
|
||
```sql | ||
yugabyte=# SELECT pg_advisory_lock(100), COUNT(*) FROM cars; | ||
``` | ||
|
||
```output | ||
ERROR: advisory locks are not yet implemented | ||
HINT: If the app doesn't need strict functionality, this error can be silenced by using the GFlag yb_silence_advisory_locks_not_supported_error. See https://github.com/yugabyte/yugabyte-db/issues/3642 for details | ||
``` | ||
|
||
**Workaround**: Implement a custom locking mechanism in the application to coordinate actions without relying on database-level advisory locks. | ||
|
||
--- | ||
|
||
### System columns is not yet supported | ||
|
||
**GitHub**: [Issue #24843](https://github.com/yugabyte/yugabyte-db/issues/24843) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are there changes to pg_locks view? eg, locktype can now also be advisory?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pg_locks changes are planned, but its not yet in the master builds.