Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on May 8, 2026. It is now read-only.

samples: Schema evolution#1469

Merged
kamalaboulhosn merged 34 commits into
googleapis:maingoogleapis/java-pubsub:mainfrom
kamalaboulhosn:masterkamalaboulhosn/java-pubsub:masterCopy head branch name to clipboard
Mar 11, 2023
Merged

samples: Schema evolution#1469
kamalaboulhosn merged 34 commits into
googleapis:maingoogleapis/java-pubsub:mainfrom
kamalaboulhosn:masterkamalaboulhosn/java-pubsub:masterCopy head branch name to clipboard

Conversation

@kamalaboulhosn

Copy link
Copy Markdown
Contributor

No description provided.

@kamalaboulhosn kamalaboulhosn requested review from a team January 23, 2023 16:48
@product-auto-label product-auto-label Bot added the size: l Pull request size is large. label Jan 23, 2023
@snippet-bot

snippet-bot Bot commented Jan 23, 2023

Copy link
Copy Markdown

Here is the summary of changes.

You are about to add 9 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

@product-auto-label product-auto-label Bot added api: pubsub Issues related to the googleapis/java-pubsub API. samples Issues that are directly related to samples. labels Jan 23, 2023
@Sita04

Sita04 commented Jan 26, 2023

Copy link
Copy Markdown

@kamalaboulhosn Samples failing test and checkstyle.

@Sita04 Sita04 added the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 26, 2023
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 26, 2023
@Sita04 Sita04 added the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 30, 2023
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 30, 2023
Comment thread samples/snippets/src/main/java/pubsub/UpdateTopicSchemaExample.java Outdated
Comment thread samples/snippets/src/main/java/pubsub/UpdateTopicSchemaExample.java Outdated
@product-auto-label product-auto-label Bot added size: xl Pull request size is extra large. and removed size: l Pull request size is large. labels Feb 23, 2023
kamalaboulhosn and others added 2 commits February 23, 2023 17:47
* samples: schema evolution

* samples: schema evolution

* Format fixes

* Fix documentation for field.

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* Add back in working asserts

* Formatting fixes

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* Version/delete fixes

* samples: schema evolution

* samples: schema evolution

* Format fixes

* Fix documentation for field.

* Add back in working asserts

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* Formatting fixes

* Version/delete fixes

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
@product-auto-label product-auto-label Bot added size: l Pull request size is large. and removed size: xl Pull request size is extra large. labels Feb 23, 2023
kamalaboulhosn and others added 5 commits February 23, 2023 18:20
* samples: schema evolution

* samples: schema evolution

* Format fixes

* Fix documentation for field.

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* Add back in working asserts

* Formatting fixes

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* Version/delete fixes

* samples: schema evolution

* samples: schema evolution

* Format fixes

* Fix documentation for field.

* Add back in working asserts

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* Formatting fixes

* Version/delete fixes

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Comment thread samples/snippets/src/main/java/pubsub/UpdateTopicSchemaExample.java Outdated
Comment thread samples/snippets/src/test/java/pubsub/SchemaIT.java
@kamalaboulhosn kamalaboulhosn requested a review from hongalex March 10, 2023 14:34
@kamalaboulhosn kamalaboulhosn added the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 10, 2023
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 10, 2023

@acocuzzo acocuzzo left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Left a couple comments re: exceptions, otherwise looks great! Thanks.

System.out.println("Committed a schema using an Avro schema:\n" + schema);
return schema;
} catch (AlreadyExistsException e) {
System.out.println(schemaName + "already exists.");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

We should probably change to FailedPrecondition/InvalidArgument/NotFound/ResourceExhausted, since we can't get AlreadyExists.


System.out.println("Committed a schema using a protobuf schema:\n" + schema);
return schema;
} catch (AlreadyExistsException e) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Same here.

@kamalaboulhosn kamalaboulhosn merged commit a28803d into googleapis:main Mar 11, 2023
quartzmo added a commit to quartzmo/google-cloud-ruby that referenced this pull request Apr 30, 2026
The design is based on SubscribeWithAvroSchemaRevisionsExample.java:
googleapis/java-pubsub#1469
quartzmo added a commit to googleapis/google-cloud-ruby that referenced this pull request May 1, 2026
…33900)

Add sample for pubsub_subscribe_avro_records_with_revisions.

The design is based on SubscribeWithAvroSchemaRevisionsExample.java:
googleapis/java-pubsub#1469
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: pubsub Issues related to the googleapis/java-pubsub API. samples Issues that are directly related to samples. size: l Pull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.