Skip to content

Add bool_and and bool_or functions#1017

Merged
lvca merged 3 commits into
ArcadeData:mainfrom
gramian:bool
Apr 8, 2023
Merged

Add bool_and and bool_or functions#1017
lvca merged 3 commits into
ArcadeData:mainfrom
gramian:bool

Conversation

@gramian

@gramian gramian commented Apr 8, 2023

Copy link
Copy Markdown
Collaborator

What does this PR do?

These changes add the SQL functions bool_and and bool_or as well as registering them. Also some tests for both new functions are added.

Related issues

#1011

Additional Notes

The test sets run rather slow, maybe I am doing something inefficiently?

Checklist

  • I have run the build using mvn clean package command
  • My unit tests cover both failure and success scenarios (currently only success)

@lvca lvca added the enhancement New feature or request label Apr 8, 2023
@lvca lvca added this to the 23.4.1 milestone Apr 8, 2023
@lvca

lvca commented Apr 8, 2023

Copy link
Copy Markdown
Member

@gramian that was super fast, thanks! Merged.

@lvca lvca merged commit 645ea7d into ArcadeData:main Apr 8, 2023
@lvca lvca self-requested a review April 8, 2023 23:56
@gramian gramian deleted the bool branch May 3, 2023 12:05
tae898 pushed a commit to humemai/arcadedb-embedded-python that referenced this pull request Jun 28, 2026
* Add bool_and and bool_or SQL functions

* Add tests, function summaries, and bool_or fix

* remove unsused import
mergify Bot added a commit that referenced this pull request Jun 29, 2026
…to 5.1.6 [skip ci]

Bumps [io.github.ascopes:protobuf-maven-plugin](https://github.com/ascopes/protobuf-maven-plugin) from 5.1.5 to 5.1.6.
Release notes

*Sourced from [io.github.ascopes:protobuf-maven-plugin's releases](https://github.com/ascopes/protobuf-maven-plugin/releases).*

> v5.1.6
> ------
>
> What's Changed
> --------------
>
> * Forbid unused imports by [`@​ascopes`](https://github.com/ascopes) in [ascopes/protobuf-maven-plugin#1017](https://redirect.github.com/ascopes/protobuf-maven-plugin/pull/1017)
> * Bump actions/checkout from 6 to 7 by [`@​dependabot`](https://github.com/dependabot)[bot] in [ascopes/protobuf-maven-plugin#1019](https://redirect.github.com/ascopes/protobuf-maven-plugin/pull/1019)
> * Added support for Solaris / illumos by [`@​siepkes`](https://github.com/siepkes) in [ascopes/protobuf-maven-plugin#1018](https://redirect.github.com/ascopes/protobuf-maven-plugin/pull/1018)
>
> New Contributors
> ----------------
>
> * [`@​siepkes`](https://github.com/siepkes) made their first contribution in [ascopes/protobuf-maven-plugin#1018](https://redirect.github.com/ascopes/protobuf-maven-plugin/pull/1018)
>
> **Full Changelog**: <ascopes/protobuf-maven-plugin@v5.1.5...v5.1.6>


Commits

* [`7bf0941`](ascopes/protobuf-maven-plugin@7bf0941) [maven-release-plugin] prepare release v5.1.6
* [`f05fe86`](ascopes/protobuf-maven-plugin@f05fe86) Merge pull request [#1018](https://redirect.github.com/ascopes/protobuf-maven-plugin/issues/1018) from siepkes/solaris-support
* [`c932d26`](ascopes/protobuf-maven-plugin@c932d26) Added support for Solaris / illumos
* [`21bb21b`](ascopes/protobuf-maven-plugin@21bb21b) Merge pull request [#1019](https://redirect.github.com/ascopes/protobuf-maven-plugin/issues/1019) from ascopes/dependabot/github\_actions/main/actions/...
* [`3c22c18`](ascopes/protobuf-maven-plugin@3c22c18) Bump actions/checkout from 6 to 7
* [`25529ac`](ascopes/protobuf-maven-plugin@25529ac) Merge pull request [#1017](https://redirect.github.com/ascopes/protobuf-maven-plugin/issues/1017) from ascopes/task/forbid-unused-imports
* [`72c4cda`](ascopes/protobuf-maven-plugin@72c4cda) Forbid unused imports
* [`a9f9e6c`](ascopes/protobuf-maven-plugin@a9f9e6c) [maven-release-plugin] prepare for next development iteration
* See full diff in [compare view](ascopes/protobuf-maven-plugin@v5.1.5...v5.1.6)
  
[![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility\_score?dependency-name=io.github.ascopes:protobuf-maven-plugin&package-manager=maven&previous-version=5.1.5&new-version=5.1.6)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.
[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)
---
Dependabot commands and options
  
You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
- `@dependabot show  ignore conditions` will show all of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants