Skip to content

Go-Ora improvements #591

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

Merged
merged 2 commits into from
Sep 11, 2024
Merged

Go-Ora improvements #591

merged 2 commits into from
Sep 11, 2024

Conversation

stle04
Copy link
Contributor

@stle04 stle04 commented Sep 9, 2024

  • Regexp variables lazy initialization
  • fix unit tests
  • fix ConvertIntervalYM_DTY with wrong unsigned integer usage
  • code formatting(gofumpt for one code style)

stle04 and others added 2 commits September 9, 2024 06:59
* fix unit tests
* fix ConvertIntervalYM_DTY with wrong unsigned integer usage
* code formatting
@sijms sijms merged commit 2067ac8 into sijms:master Sep 11, 2024
0 of 3 checks passed
@kopaygorodsky
Copy link

Hey, I've stumbled upon this PR due to an error I'm facing now.
So I have my custom type

type UUID []byte

// Value implements sql.Valuer so that UUIDs can be written to databases
// transparently. This method returns a byte slice representation of uuid
func (u UUID) Value() (driver.Value, error) {
	return []byte(u), nil
}

Column type is RAW(16), I'm trying to execute INSERT INTO and one of my params is ID of type I mentioned above.
Execution fails with this error parameter require array value which I checked is wrong in this case as my value is in fact an "array".

CleanShot 2025-03-04 at 02 05 34@2x

@kopaygorodsky
Copy link

oh, nvm, I'm wrong about this particular PR, the issue is not related to this one.

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.

3 participants