Skip to content

Feat oci8 modify column#7

Open
vabshere wants to merge 26 commits intomisterEggroll:masterfrom
skit-ai:feat-oci8-modify-column
Open

Feat oci8 modify column#7
vabshere wants to merge 26 commits intomisterEggroll:masterfrom
skit-ai:feat-oci8-modify-column

Conversation

@vabshere
Copy link

Make sure these boxes checked before submitting your pull request.

  • Do only one thing
  • No API-breaking changes
  • New code/logic commented & tested

For significant changes like big bug fixes, new features, please open an issue to make an agreement on an implementation design/plan first before starting it.

What did this pull request do?

akshaydeshraj and others added 26 commits August 16, 2019 16:17
add: Fn to remove constraint from a table
…limit

add: Fn to determine dialect-specific byte insertion limit
* If records go beyond 1000000, the ID returned by the oracle driver is of type float64 instead of uint(uint64)
* To mitigate this issue, introducing the `primary` struct
* The `primary` struct implements the `Scanner` interface to convert float64 and int64 into a unit ID
* The `primary` struct implements the `Valuer` interface to return the uint value of the ID it encapsulates
* Introducing the `ORM` struct which makes use of the `primary` struct to encapsulate the ID returned by the DB
* This is to mitigate the scenario where the reference to the rowid of the oracle entry is returned as an int64.
* This int64 is converted to uint internally by the Scan() implementation of the `primary` struct
* Hence, we need to access that converted value by checking if the primary field is of type `primary`
refactor: Export `Primary` struct outside gorm package
refactor: Remove primary key and foreign key custom structs
refactor: Use primaryField.Field.Uint to get uint when resolving row ID
feat: add support for setting nullable
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

Successfully merging this pull request may close these issues.

4 participants