Skip to content

fix!: [discoveryengine] http_uri fix for method ListChunks in service ChunkService#10508

Closed
gcf-owl-bot[bot] wants to merge 12 commits into
mainfrom
owl-bot-copy-java-discoveryengine
Closed

fix!: [discoveryengine] http_uri fix for method ListChunks in service ChunkService#10508
gcf-owl-bot[bot] wants to merge 12 commits into
mainfrom
owl-bot-copy-java-discoveryengine

Conversation

@gcf-owl-bot

@gcf-owl-bot gcf-owl-bot Bot commented Mar 9, 2024

Copy link
Copy Markdown
Contributor
  • Regenerate this pull request now.

PiperOrigin-RevId: 614683212

Source-Link: googleapis/googleapis@716a939

Source-Link: https://github.com/googleapis/googleapis-gen/commit/a54c0126868455148c339ce88209dca049e84171
Copy-Tag: eyJwIjoiamF2YS1kaXNjb3ZlcnllbmdpbmUvLk93bEJvdC55YW1sIiwiaCI6ImE1NGMwMTI2ODY4NDU1MTQ4YzMzOWNlODgyMDlkY2EwNDllODQxNzEifQ==

BEGIN_NESTED_COMMIT
feat: [discoveryengine] support acl-based and blended search
feat: support chunk-based document search
feat: support search adaptor based search
feat: support batch documents purge with GCS input
feat: add document processing config services
feat: add serving config services
feat: add billing estimation services
feat: add suggestion deny list import/purge services
docs: keep the API doc up-to-date with recent changes

PiperOrigin-RevId: 614160461

Source-Link: googleapis/googleapis@b553626

Source-Link: https://github.com/googleapis/googleapis-gen/commit/4394e3704d4d04697ef54d16ffb7d5057ab3a2fe
Copy-Tag: eyJwIjoiamF2YS1kaXNjb3ZlcnllbmdpbmUvLk93bEJvdC55YW1sIiwiaCI6IjQzOTRlMzcwNGQ0ZDA0Njk3ZWY1NGQxNmZmYjdkNTA1N2FiM2EyZmUifQ==
END_NESTED_COMMIT
BEGIN_NESTED_COMMIT
feat: [discoveryengine] support blended search
feat: support boost in multi-turn search
feat: add document processing config services
feat: add search tuning services
feat: allow setting schema on schema creation
docs: keep the API doc up-to-date with recent changes

PiperOrigin-RevId: 614046598

Source-Link: googleapis/googleapis@d66613f

Source-Link: https://github.com/googleapis/googleapis-gen/commit/3d656450d3a676ceab7078c673f2e05243e01d8f
Copy-Tag: eyJwIjoiamF2YS1kaXNjb3ZlcnllbmdpbmUvLk93bEJvdC55YW1sIiwiaCI6IjNkNjU2NDUwZDNhNjc2Y2VhYjcwNzhjNjczZjJlMDUyNDNlMDFkOGYifQ==
END_NESTED_COMMIT

feat: support boost in multi-turn search
feat: add document processing config services
feat: add search tuning services
feat: allow setting schema on schema creation
docs: keep the API doc up-to-date with recent changes

PiperOrigin-RevId: 614046598

Source-Link: googleapis/googleapis@d66613f

Source-Link: https://github.com/googleapis/googleapis-gen/commit/3d656450d3a676ceab7078c673f2e05243e01d8f
Copy-Tag: eyJwIjoiamF2YS1kaXNjb3ZlcnllbmdpbmUvLk93bEJvdC55YW1sIiwiaCI6IjNkNjU2NDUwZDNhNjc2Y2VhYjcwNzhjNjczZjJlMDUyNDNlMDFkOGYifQ==
@trusted-contributions-gcf trusted-contributions-gcf Bot added kokoro:force-run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Mar 9, 2024
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 9, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Mar 9, 2024
@gcf-owl-bot gcf-owl-bot Bot marked this pull request as ready for review March 9, 2024 01:16
@gcf-owl-bot gcf-owl-bot Bot requested review from a team March 9, 2024 01:16
@trusted-contributions-gcf trusted-contributions-gcf Bot added kokoro:force-run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Mar 9, 2024
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 9, 2024
@snippet-bot

snippet-bot Bot commented Mar 9, 2024

Copy link
Copy Markdown

Here is the summary of changes.

You are about to add 74 region tags.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Mar 9, 2024
@trusted-contributions-gcf trusted-contributions-gcf Bot added kokoro:force-run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Mar 9, 2024
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 9, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Mar 9, 2024
feat: support chunk-based document search
feat: support search adaptor based search
feat: support batch documents purge with GCS input
feat: add document processing config services
feat: add serving config services
feat: add billing estimation services
feat: add suggestion deny list import/purge services
docs: keep the API doc up-to-date with recent changes

PiperOrigin-RevId: 614160461

Source-Link: googleapis/googleapis@b553626

Source-Link: https://github.com/googleapis/googleapis-gen/commit/4394e3704d4d04697ef54d16ffb7d5057ab3a2fe
Copy-Tag: eyJwIjoiamF2YS1kaXNjb3ZlcnllbmdpbmUvLk93bEJvdC55YW1sIiwiaCI6IjQzOTRlMzcwNGQ0ZDA0Njk3ZWY1NGQxNmZmYjdkNTA1N2FiM2EyZmUifQ==
@gcf-owl-bot gcf-owl-bot Bot changed the title feat: [discoveryengine] support blended search feat: [discoveryengine] support acl-based and blended search Mar 9, 2024
@trusted-contributions-gcf trusted-contributions-gcf Bot added kokoro:force-run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Mar 9, 2024
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 9, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Mar 9, 2024
@trusted-contributions-gcf trusted-contributions-gcf Bot added kokoro:force-run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Mar 9, 2024
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 9, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Mar 9, 2024
@trusted-contributions-gcf trusted-contributions-gcf Bot added kokoro:force-run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Mar 9, 2024
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 9, 2024
@yoshi-kokoro yoshi-kokoro removed kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Mar 9, 2024
mockService.addResponse(expectedResponse);

String parent =
"projects/project-3187/locations/location-3187/dataStores/dataStore-3187/branches/branche-3187";

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There may be an issue with the parent path where part of the segment is missing as the rpc ListChunks has "/v1alpha/{parent=projects/*/locations/*/dataStores/*/branches/*}/documents/*/chunks" instead of "/v1alpha/{parent=projects/*/locations/*/dataStores/*/branches/*/documents/*}/chunks"

https://github.com/googleapis/googleapis/blob/569bf5b0541481388a3763157c5447e65839743f/google/cloud/discoveryengine/v1alpha/chunk_service.proto#L54-L63

This is also causing an issue for python client library generation. See googleapis/google-cloud-python#12433 (comment)

Googlers see b/328843326 and b/328869136

@diegomarquezp diegomarquezp added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Mar 11, 2024
@diegomarquezp

Copy link
Copy Markdown
Contributor

We will use #10503 once it's merged for all library source code changes

PiperOrigin-RevId: 614683212

Source-Link: googleapis/googleapis@716a939

Source-Link: https://github.com/googleapis/googleapis-gen/commit/a54c0126868455148c339ce88209dca049e84171
Copy-Tag: eyJwIjoiamF2YS1kaXNjb3ZlcnllbmdpbmUvLk93bEJvdC55YW1sIiwiaCI6ImE1NGMwMTI2ODY4NDU1MTQ4YzMzOWNlODgyMDlkY2EwNDllODQxNzEifQ==
@gcf-owl-bot gcf-owl-bot Bot changed the title feat: [discoveryengine] support acl-based and blended search fix!: [discoveryengine] http_uri fix for method ListChunks in service ChunkService Mar 11, 2024
@trusted-contributions-gcf trusted-contributions-gcf Bot added kokoro:force-run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Mar 11, 2024
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 11, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Mar 11, 2024
@trusted-contributions-gcf trusted-contributions-gcf Bot added kokoro:force-run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Mar 11, 2024
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 11, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Mar 11, 2024
@trusted-contributions-gcf trusted-contributions-gcf Bot added kokoro:force-run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Mar 11, 2024
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 11, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Mar 11, 2024
@blakeli0 blakeli0 closed this Mar 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge Indicates a pull request not ready for merge, due to either quality or timing. owl-bot-copy

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants