-
Notifications
You must be signed in to change notification settings - Fork 99
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
Epic 8349 certify s3/issue 8556 s3 v2 fields #85
Open
johnny-schmidt
wants to merge
30
commits into
allegro:master
Choose a base branch
from
airbytehq:epic-8349-certify-s3/issue-8556-s3-v2-fields
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Epic 8349 certify s3/issue 8556 s3 v2 fields #85
johnny-schmidt
wants to merge
30
commits into
allegro:master
from
airbytehq:epic-8349-certify-s3/issue-8556-s3-v2-fields
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Add gradle action * Move file * Use gradle 6.5 * Use java 14 * Update validation action * Specify ref and repo * Update gradle commands * Set fetch-depth * Use java 11 * Update badge link * Run gradle build
* Add field name map * Set up unit test * Support additional properties map * Test more primitive field types * Run name transformation consistently
* Use maven-publish plugin * Remove com.bmuschko.nexus * Use me.champeau.gradle.jmh version 0.5.3 * Replace compile with implementation * Use gradle 7.2 * Remove maven-publish * Update readme * Add missing dependency
* Build with java 14 * Use groovy 2.5.10
* Delete travis ci * Update readme to add build changes
* Specify input and output extra props fields * Update README
* support date/time logical types in avro * PR fixes * remove star import * Format comments Co-authored-by: vmaltsev <[email protected]> Co-authored-by: Liren Tu <[email protected]>
* Reorder test cases * Force value to string * Fix name * Add documentation in README
* Enforce string only when string is the only item type * Update readme
* Add PMD to prevent system printout * Add pr template * Update pr template
* Add caching in ci * Log offending values * Update readme
* Bump gradle wrapper * Clean up cache key * Fix non-nullable field * Upgrade spock and groovy * Use java 17 * Remove tool chain config (cannot be set together with source target compatibility * Set java version for all projects * Try toolchain in base * Try toolchain in projects * Revert build.gradle changes * Avoid using java 17 method
* Support Era and string Numbers * Update Readme * Modify exception formatting * Added infinity support for double type * Merge date and datetime formatters into one * Added onValidStringNumber * Support base56 decode * Added test to verify Infinity and NaN is not supported for avro int type
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.