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.

skyflow-bharti and others added 30 commits August 21, 2025 15:08
…sation' into bharti/SK-2258-implement-insert-public-interface-for-v-3-flow-db-in-java-sdk
@skyflow-bharti skyflow-bharti changed the base branch from skyflow-vivek/SK-2269-add-get-retry-records-util-in-insert-response to skyflow-vivek/SK-2259-immutable-skyflow-client-initialisation August 27, 2025 05:12
@skyflow-vivek skyflow-vivek changed the base branch from skyflow-vivek/SK-2259-immutable-skyflow-client-initialisation to v3 August 27, 2025 06:51
skyflow-bharti and others added 27 commits August 27, 2025 13:29
- Updated return type from List of Map to ArrayList of HashMap
…rt-public-interface-for-v-3-flow-db-in-java-sdk

SK-2258 fix error response records
…rt-public-interface-for-v-3-flow-db-in-java-sdk

SK-2258 synchronized errors
@skyflow-vivek skyflow-vivek merged commit 742a05b into v3 Sep 8, 2025
4 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