Skip to content

samples: Schema evolution#1469

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

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

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
Copy link
Copy Markdown

snippet-bot Bot commented Jan 23, 2023

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
Copy link
Copy Markdown

Sita04 commented Jan 26, 2023

@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
Copy link
Copy Markdown

@acocuzzo acocuzzo left a comment

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 join this conversation on GitHub. Already have an account? Sign in to comment

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