Skip to content

Conversation

@docbacardi
Copy link
Contributor

Adding defaults allows parsing a string without time information to a date + time object.

What does this PR do?

Use the date() function with just a date, i.e. without time fields: SELECT date('2012-07-02').

Motivation

Specifying a date without time fields raised an exception.

Related issues

This patch fixes #1232 .

Additional Notes

Still working on test cases, sorry.

Checklist

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

Adding defaults allows parsing a string without time information to a
date + time object.
@lvca
Copy link
Contributor

lvca commented Sep 26, 2023

Thanks @docbacardi!

@lvca lvca merged commit 805631d into ArcadeData:main Sep 26, 2023
@lvca lvca added this to the 23.9.1 milestone Sep 26, 2023
@lvca lvca added bug Something isn't working fixed / implemented labels Sep 26, 2023
mergify bot added a commit that referenced this pull request Sep 8, 2025
… to 2.19.0 [skip ci]

Bumps [org.codehaus.mojo:versions-maven-plugin](https://github.com/mojohaus/versions) from 2.18.0 to 2.19.0.
Release notes

*Sourced from [org.codehaus.mojo:versions-maven-plugin's releases](https://github.com/mojohaus/versions/releases).*

> 2.19.0
> ------
>
> 💥 Breaking changes
> -----------------------
>
> * Removed numeric and mercury comparators, further refactoring ([#1222](https://redirect.github.com/mojohaus/versions/pull/1222)) [`@​andrzejj0`](https://github.com/andrzejj0)
>
> 🚀 New features and improvements
> -------------------------------
>
> * Parallel version updates ([#1207](https://redirect.github.com/mojohaus/versions/pull/1207)) [`@​andrzejj0`](https://github.com/andrzejj0)
> * Fixes [#1140](https://redirect.github.com/mojohaus/versions/issues/1140): Added showVersionless (default true) ([#1187](https://redirect.github.com/mojohaus/versions/pull/1187)) [`@​andrzejj0`](https://github.com/andrzejj0)
>
> 🐛 Bug Fixes
> -----------
>
> * Resolves [#1227](https://redirect.github.com/mojohaus/versions/issues/1227): revert working in case of a non-standard "pom.xml" filename ([#1263](https://redirect.github.com/mojohaus/versions/pull/1263)) [`@​andrzejj0`](https://github.com/andrzejj0)
> * Resolves [#1255](https://redirect.github.com/mojohaus/versions/issues/1255): set-property special case when a property has no value ([#1261](https://redirect.github.com/mojohaus/versions/pull/1261)) [`@​andrzejj0`](https://github.com/andrzejj0)
> * Resolves [#1251](https://redirect.github.com/mojohaus/versions/issues/1251): nextSnapshotIndexToIncrement should zeros on the right ([#1262](https://redirect.github.com/mojohaus/versions/pull/1262)) [`@​andrzejj0`](https://github.com/andrzejj0)
> * Fixed #1233- set-property in profile without module property ([#1234](https://redirect.github.com/mojohaus/versions/pull/1234)) [`@​mcarlett`](https://github.com/mcarlett)
> * Fix closing code tags ([#1206](https://redirect.github.com/mojohaus/versions/pull/1206)) [`@​pzygielo`](https://github.com/pzygielo)
> * Resolves [#1197](https://redirect.github.com/mojohaus/versions/issues/1197): Handling a no-version case in dependencyManagement for CompareDependenciesMojo ([#1198](https://redirect.github.com/mojohaus/versions/pull/1198)) [`@​andrzejj0`](https://github.com/andrzejj0)
> * Fixed the failed release build ([#1194](https://redirect.github.com/mojohaus/versions/pull/1194)) [`@​andrzejj0`](https://github.com/andrzejj0)
> * Fixed [#1191](https://redirect.github.com/mojohaus/versions/issues/1191): Processes (all) profiles from original model if transitive is false ([#1193](https://redirect.github.com/mojohaus/versions/pull/1193)) [`@​andrzejj0`](https://github.com/andrzejj0)
> * Fixes [#1140](https://redirect.github.com/mojohaus/versions/issues/1140) for reports ([#1188](https://redirect.github.com/mojohaus/versions/pull/1188)) [`@​andrzejj0`](https://github.com/andrzejj0)
> * Fixes [#1140](https://redirect.github.com/mojohaus/versions/issues/1140): Added showVersionless (default true) ([#1187](https://redirect.github.com/mojohaus/versions/pull/1187)) [`@​andrzejj0`](https://github.com/andrzejj0)
> * Fixes [#1182](https://redirect.github.com/mojohaus/versions/issues/1182) and [#1184](https://redirect.github.com/mojohaus/versions/issues/1184): Bug fixes ([#1185](https://redirect.github.com/mojohaus/versions/pull/1185)) [`@​andrzejj0`](https://github.com/andrzejj0)
>
> 👻 Maintenance
> -------------
>
> * Remove manual default from docs for parameters ([#1264](https://redirect.github.com/mojohaus/versions/pull/1264)) [`@​slawekjaranowski`](https://github.com/slawekjaranowski)
> * Use default configuration for release-drafter ([#1256](https://redirect.github.com/mojohaus/versions/pull/1256)) [`@​slawekjaranowski`](https://github.com/slawekjaranowski)
> * Bump project version to 2.19.0-SNAPSHOT ([#1237](https://redirect.github.com/mojohaus/versions/pull/1237)) [`@​slawekjaranowski`](https://github.com/slawekjaranowski)
> * Exposing the problem causing artifact if Resolver throws an exception ([#1224](https://redirect.github.com/mojohaus/versions/pull/1224)) [`@​andrzejj0`](https://github.com/andrzejj0)
> * Site fix for rule-2.1.0.xsd and rule-2.0.0.xsd; getOutputPath() for reports ([#1223](https://redirect.github.com/mojohaus/versions/pull/1223)) [`@​andrzejj0`](https://github.com/andrzejj0)
> * Removed numeric and mercury comparators, further refactoring ([#1222](https://redirect.github.com/mojohaus/versions/pull/1222)) [`@​andrzejj0`](https://github.com/andrzejj0)
> * Pom helper refactoring ([#1221](https://redirect.github.com/mojohaus/versions/pull/1221)) [`@​andrzejj0`](https://github.com/andrzejj0)
> * PR Automation only on close event ([#1220](https://redirect.github.com/mojohaus/versions/pull/1220)) [`@​slawekjaranowski`](https://github.com/slawekjaranowski)
> * Refactoring: ArtifactFactory and ArtifactVersionService ([#1218](https://redirect.github.com/mojohaus/versions/pull/1218)) [`@​andrzejj0`](https://github.com/andrzejj0)
> * Versions refactor 2 ([#1204](https://redirect.github.com/mojohaus/versions/pull/1204)) [`@​andrzejj0`](https://github.com/andrzejj0)
> * Refactoring: coalesce use-*-* goals. ([#1202](https://redirect.github.com/mojohaus/versions/pull/1202)) [`@​andrzejj0`](https://github.com/andrzejj0)
> * PR Automation - must use pull\_request\_target ([#1205](https://redirect.github.com/mojohaus/versions/pull/1205)) [`@​slawekjaranowski`](https://github.com/slawekjaranowski)
> * Add PR Automation ([#1201](https://redirect.github.com/mojohaus/versions/pull/1201)) [`@​slawekjaranowski`](https://github.com/slawekjaranowski)
>
> 📦 Dependency updates
> --------------------
>
> * Bump actions/stale from 9 to 10 ([#1265](https://redirect.github.com/mojohaus/versions/pull/1265)) @[dependabot[bot]](https://github.com/apps/dependabot)
> * Bump org.codehaus.plexus:plexus-i18n from 1.0-beta-10 to 1.0.0 ([#1260](https://redirect.github.com/mojohaus/versions/pull/1260)) @[dependabot[bot]](https://github.com/apps/dependabot)
> * Bump org.codehaus.plexus:plexus-archiver from 4.10.0 to 4.10.1 ([#1259](https://redirect.github.com/mojohaus/versions/pull/1259)) @[dependabot[bot]](https://github.com/apps/dependabot)
> * Bump byteBuddyVersion from 1.17.6 to 1.17.7 ([#1258](https://redirect.github.com/mojohaus/versions/pull/1258)) @[dependabot[bot]](https://github.com/apps/dependabot)
> * Bump org.codehaus.mojo:mojo-parent from 92 to 93 ([#1257](https://redirect.github.com/mojohaus/versions/pull/1257)) @[dependabot[bot]](https://github.com/apps/dependabot)
> * Bump org.apache.commons:commons-text from 1.13.1 to 1.14.0 ([#1254](https://redirect.github.com/mojohaus/versions/pull/1254)) @[dependabot[bot]](https://github.com/apps/dependabot)
> * Bump commons-codec:commons-codec from 1.18.0 to 1.19.0 ([#1253](https://redirect.github.com/mojohaus/versions/pull/1253)) @[dependabot[bot]](https://github.com/apps/dependabot)

... (truncated)


Commits

* [`b625a90`](mojohaus/versions@b625a90) [maven-release-plugin] prepare release 2.19.0
* [`ba67cf1`](mojohaus/versions@ba67cf1) Bump actions/stale from 9 to 10
* [`28ac80d`](mojohaus/versions@28ac80d) Remove manual default from docs for parameters
* [`0ce0fba`](mojohaus/versions@0ce0fba) Resolves [#1227](https://redirect.github.com/mojohaus/versions/issues/1227): revert working in case a non-standard "pom.xml" file name is ...
* [`d547a03`](mojohaus/versions@d547a03) Resolves [#1255](https://redirect.github.com/mojohaus/versions/issues/1255): set-property special case when the property has no value ([#1261](https://redirect.github.com/mojohaus/versions/issues/1261))
* [`e99daf3`](mojohaus/versions@e99daf3) Resolves [#1251](https://redirect.github.com/mojohaus/versions/issues/1251): nextSnapshotIndexToIncrement should zero versions components ...
* [`7a05285`](mojohaus/versions@7a05285) Bump org.codehaus.plexus:plexus-i18n from 1.0-beta-10 to 1.0.0 ([#1260](https://redirect.github.com/mojohaus/versions/issues/1260))
* [`7444785`](mojohaus/versions@7444785) Bump org.codehaus.plexus:plexus-archiver from 4.10.0 to 4.10.1
* [`e3584a2`](mojohaus/versions@e3584a2) Bump byteBuddyVersion from 1.17.6 to 1.17.7
* [`129aad2`](mojohaus/versions@129aad2) Bump org.codehaus.mojo:mojo-parent from 92 to 93
* Additional commits viewable in [compare view](mojohaus/versions@2.18.0...2.19.0)
  
[![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility\_score?dependency-name=org.codehaus.mojo:versions-maven-plugin&package-manager=maven&previous-version=2.18.0&new-version=2.19.0)](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 merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
- `@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

bug Something isn't working fixed / implemented

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Creating date objects with the date() function

2 participants