Skip to content

Types: Add method ObjectArray.as_generic#23

Merged
amotl merged 1 commit into
mainfrom
amo/objectarray-asgeneric
Jun 4, 2026
Merged

Types: Add method ObjectArray.as_generic#23
amotl merged 1 commit into
mainfrom
amo/objectarray-asgeneric

Conversation

@amotl
Copy link
Copy Markdown
Contributor

@amotl amotl commented Jan 15, 2024

About

AFAIK, this patch is needed for improved reverse type lookups, in this case for CrateDB's ARRAY type.

References

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 2, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: dabbaa89-2fd4-4bef-a85d-e98a2f30c868

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch amo/objectarray-asgeneric

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@amotl amotl force-pushed the amo/objectarray-asgeneric branch 2 times, most recently from 1c2c212 to adff60b Compare June 2, 2026 19:21
@amotl amotl requested review from bgunebakan and matriv June 2, 2026 19:31
@amotl amotl marked this pull request as ready for review June 2, 2026 19:32
@amotl amotl requested review from kneth and removed request for matriv June 2, 2026 19:32
self.operator = operator


# TODO: Should this be inherited from PostgreSQL's
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please create an issue about this

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AFAIK, it is needed for reverse type lookups.
@amotl amotl force-pushed the amo/objectarray-asgeneric branch from adff60b to b8a21d8 Compare June 4, 2026 21:08
@amotl amotl merged commit 730bca6 into main Jun 4, 2026
38 checks passed
@amotl amotl deleted the amo/objectarray-asgeneric branch June 4, 2026 21:16
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.

2 participants