Skip to content

Conversation

@skyflow-bharti
Copy link
Collaborator

Start with a concise summary of the PR. The first three sections are required. The questions present in each section is there to help you guide you what to add. They are meant to be overwritten by your comments.

Why

  • Why are you making the change?
  • What is the underlying issue that you are trying to case, in case of fix?
  • Why is it needed by the feature you are working on?
  • What is the intent behind making the change?

Goal

  • What is the intended outcome?
  • What part of the feature should start working?
  • What are the non-goals or will be covered in future PR?

Testing

  • How was the code tested?
  • If you haven't written unit tests, why?
  • What more testing is needed? Do you intend to manually test it after deployment?
  • Do you have any concerns if this changed is released to prod?

Tech debt

  • Is the PR adding to tech debt in any way?
  • Are you addressing some Tech debt in this PR?
  • If both the above are false, feel free to remove this section.

saileshwar-skyflow and others added 19 commits August 11, 2025 13:48
- Fixed some errors in overall project
- Updated POM configuration
- Fixed some failing unit tests
- Refactored more code into common
- Added more unit tests for V3
…sation' into bharti/SK-2258-implement-insert-public-interface-for-v-3-flow-db-in-java-sdk
- Add getRecordsToRetry implementation in InsertResponse
- Refactored code for better code quality
…ert-response' into bharti/SK-2258-implement-insert-public-interface-for-v-3-flow-db-in-java-sdk
- Updated return type from List of Map to ArrayList of HashMap
@skyflow-bharti skyflow-bharti merged commit 540cf6f into v3-release/25.8.2 Aug 27, 2025
2 checks passed
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.

4 participants