Skip to content
This repository was archived by the owner on Aug 13, 2026. It is now read-only.

samples: Schema evolution - #1469

Merged
kamalaboulhosn merged 34 commits into
googleapis:mainfrom
kamalaboulhosn:master
Mar 11, 2023
Merged

samples: Schema evolution#1469
kamalaboulhosn merged 34 commits into
googleapis:mainfrom
kamalaboulhosn:master

Conversation

@kamalaboulhosn

Copy link
Copy Markdown
Contributor

No description provided.

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

snippet-botBot 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-labelproduct-auto-labelBot added api: pubsub Issues related to the googleapis/java-pubsub API. samples Issues that are directly related to samples. labels Jan 23, 2023
@Sita04

Copy link
Copy Markdown

@kamalaboulhosn Samples failing test and checkstyle.

@Sita04Sita04 added the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 26, 2023
@gcf-owl-botgcf-owl-botBot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 26, 2023
@Sita04Sita04 added the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 30, 2023
@gcf-owl-botgcf-owl-botBot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 30, 2023
Comment threadsamples/snippets/src/main/java/pubsub/UpdateTopicSchemaExample.java Outdated
Comment threadsamples/snippets/src/main/java/pubsub/UpdateTopicSchemaExample.java Outdated
@product-auto-labelproduct-auto-labelBot added size: xl Pull request size is extra large. and removed size: l Pull request size is large. labels Feb 23, 2023
kamalaboulhosnand 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-labelproduct-auto-labelBot added size: l Pull request size is large. and removed size: xl Pull request size is extra large. labels Feb 23, 2023
kamalaboulhosnand 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 threadsamples/snippets/src/main/java/pubsub/UpdateTopicSchemaExample.java Outdated
Comment threadsamples/snippets/src/test/java/pubsub/SchemaIT.java
@kamalaboulhosnkamalaboulhosn added the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 10, 2023
@gcf-owl-botgcf-owl-botBot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 10, 2023

@acocuzzoacocuzzo 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:mainMar 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 freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: pubsubIssues related to the googleapis/java-pubsub API.samplesIssues that are directly related to samples.size: lPull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@kamalaboulhosn@Sita04@hongalex@acocuzzo