Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Release GA Java Driver (1.0) #187

Merged
merged 219 commits into from
Nov 14, 2024
Merged

Release GA Java Driver (1.0) #187

merged 219 commits into from
Nov 14, 2024

Conversation

findgriffin
Copy link
Contributor

@findgriffin findgriffin commented Nov 13, 2024 •
edited
Loading

Description

Merge main into release, and update version in gradle to 1.0.

This release includes support for Fauna Event Streaming, and Event Feeds (beta).

Motivation and context

The Fauna FQL v10 JVM driver is now generally available!

How was the change tested?

We have:

  1. Unit tests (84% line coverage currently).
  2. End to end tests
  3. A Java Sample app
  4. A couple of bug bashes.

Screenshots (if appropriate):

Change types

    • Bug fix (non-breaking change that fixes an issue)
    • New feature (non-breaking change that adds functionality)
    • Breaking change (backwards-incompatible fix or feature)

Checklist:

    • My code follows the code style of this project.
    • My change requires a change to Fauna documentation.
    • My change requires a change to the README, and I have updated it accordingly.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

David Griffin and others added 30 commits September 11, 2024 11:54
gradle.properties: Update version to 0.2.0-B2-SNAPSHOT.
7fcacc6
@findgriffin
Merge pull request #141 from fauna/merge-0.2.0-B1
d3661fc 
Merge 0.2.0-B1 back into main
@jrodewig
[DOCS-3385] README: Add getType() ex for streaming
9bc2138
Implement parsing of ErrorInfo using JsonParser.
a299a64
Refactor error parsing.
39bcd08 
  * Add parse(..) methods to ConstraintFailure and ErrorInfo.
  * Add ErrorInfoTest.
  * Add E2EStreamingTest.handleStreamError.
@findgriffin
Merge pull request #142 from fauna/add-event-type
7004bf1 
[DOCS-3385] README: Add `getType()` ex for streaming
Handle abort data.
8b00d12
Add @deprecated annotation.
3686f9e
Avoid NullPointerException.
ba7d064
Mark ConstraintFailure constructor as deprecated.
dd7b9e7
Rename E2EConstraintTest -> E2EErrorHandlingTest and add testAbortApi.
054ba95
PR feedback: fix some exceptions.
1578824
Fix: remove unused imports.
6acec51
@findgriffin
Merge pull request #143 from fauna/implement-error-parsers
ca80aef 
[Implement-error-parsers] Refactor to use JsonParser instead of ObjectMapper for parsing errors.
Refactor to use parser and add QueryTags.
f83f8b8
Start refactoring tests.
1a5db51
Fix 23 tests and Abort data.
f7b9436
Remove handleResponse method and associated test fixes.
e8613de
Fix missing stats and stop using QueryResponseWire in tests.
3e11b77
Remove methods handleErrorResponse and QueryFailure constructor.
7a8ce52
Mark QueryResponseWire as @deprecated and remove some imports.
152c716
Refactor ConstraintFailure.
66aaa62 
  * Remove unused constructor.
  * Re-enable constraint failure E2E test.
  * Add new equality methods.
Fix and refactor QueryTags parsing.
784af61
Add E2EQueryTest.query_withTags.
8587809
A couple of nips/tucks I noticed when reading the diff.
2f5a4d3
Fix test I didn't run before pushing.
ae1edce
@findgriffin @pnwpedro
QueryStats: Replace ClientException with ClientResponseException
f4a43f3 
Co-authored-by: Lucas Pedroza <40873230+pnwpedro@users.noreply.github.com>
@findgriffin @pnwpedro
Split complex line of code into two lines.
d74aa54 
Co-authored-by: Lucas Pedroza <40873230+pnwpedro@users.noreply.github.com>
@findgriffin @pnwpedro
Remove stray call to new.
5e218ee 
Co-authored-by: Lucas Pedroza <40873230+pnwpedro@users.noreply.github.com>
PR feedback: Return null instead of throwing.
ae3b305
findgriffin and others added 25 commits November 11, 2024 10:51
@findgriffin @jrodewig
Update src/main/java/com/fauna/constants/ResponseFields.java
d74252e 
Co-authored-by: James Rodewig <james.rodewig@fauna.com>
@findgriffin @jrodewig
Update src/main/java/com/fauna/constants/ErrorMessages.java
856d689 
Co-authored-by: James Rodewig <james.rodewig@fauna.com>
@findgriffin @jrodewig
Update src/main/java/com/fauna/constants/ResponseFields.java
a74f5ee 
Co-authored-by: James Rodewig <james.rodewig@fauna.com>
@findgriffin @jrodewig
Apply James' suggestions with links
81f27b5 
Co-authored-by: James Rodewig <james.rodewig@fauna.com>
@findgriffin @jrodewig
Apply suggestions from code review
7b7d86f 
Co-authored-by: James Rodewig <james.rodewig@fauna.com>
@findgriffin @jrodewig
Update src/main/java/com/fauna/event/package-info.java
26ea423 
Co-authored-by: James Rodewig <james.rodewig@fauna.com>
Fix typo in GitHub URL.
c0f2997
@findgriffin
Merge pull request #180 from fauna/BT-4262-github-typo
cc38a39 
Fix typo in GitHub URL.
@pnwpedro
Merge pull request #176 from fauna/com.fauna.event-javadocs
da40bb5 
Fix checkstyle issues in com.fauna.event
@pnwpedro
Merge pull request #178 from fauna/com.fauna.constants-javadocs
5056397 
Fix checkstyle issues in com.fauna.constants
@jrodewig
Update README for GA release
8f0f272
@pnwpedro
Fix checkstyle issues in com.fauna.codec
9a73c93
@jrodewig
Review feedback
64d2c73
@pnwpedro
Merge pull request #181 from fauna/com.fauna.codec-javadocs
77b1727 
Fix checkstyle issues in com.fauna.codec
@pnwpedro
Fix checkstyle issues in com.fauna.client
d5ab787
@pnwpedro
Merge pull request #182 from fauna/com.fauna.client-javadocs
2cf49bd 
Fix checkstyle issues in com.fauna.client
@pnwpedro
Handle unmaterialized sets in PageCodec
597a1f9
@findgriffin
Merge pull request #183 from fauna/decode-unmaterialized-sets
953f8e4 
Handle unmaterialized sets in PageCodec
Remove references to EventSourceResponse.
5c518ae
Delete EventSourceResponse.java.
26336e1
@pnwpedro
Apply suggestions from code review
1555cc3
@pnwpedro
Merge pull request #184 from fauna/BT-5321-eventsource
17ee17a 
[BT-5321-eventsource] Refactor: Delete EventSourceResponse.java, as it duplicates EventSource.java
@jrodewig
[DOCS-3726] Replace EventSourceResponse with EventSource
e497d52
@jrodewig
Merge pull request #186 from fauna/readme-evt-source-replace
e874908 
[DOCS-3726] Replace `EventSourceResponse` with `EventSource`
gradle.properties: set version to 1.0.
1eafa70
@findgriffin findgriffin requested a review from a team November 13, 2024 23:31
@findgriffin findgriffin merged commit 76a8ba5 into release Nov 14, 2024
1 check passed
@findgriffin findgriffin deleted the release-1.0 branch November 14, 2024 00:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adambollen adambollen adambollen approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@findgriffin @adambollen @jrodewig @pnwpedro