Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: Make error message more clear about where ordering must be enabled when publishing #293

Merged
merged 11 commits into from Jul 24, 2020
Merged

fix: Make error message more clear about where ordering must be enabled when publishing #293

Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
2b50adb
feat: Add flow control support to publisher
kamalaboulhosn Mar 24, 2020
6bb47bc
make suggested fixes
kamalaboulhosn Mar 25, 2020
0ffda7c
Merge remote-tracking branch 'upstream/master'
kamalaboulhosn May 2, 2020
9c113c3
chore: Remove note that ordering keys requires enablements.
kamalaboulhosn May 2, 2020
2a295c1
Merge remote-tracking branch 'upstream/master'
kamalaboulhosn Jun 21, 2020
6947740
feat: Add support for server-side flow control
kamalaboulhosn Jun 21, 2020
fb08aa3
Revert "chore: Remove note that ordering keys requires enablements."
kamalaboulhosn Jun 21, 2020
304059f
Merge remote-tracking branch 'upstream/master'
kamalaboulhosn Jun 24, 2020
c9c2028
fix: Fix import order
kamalaboulhosn Jun 24, 2020
7b50aa1
Merge remote-tracking branch 'upstream/master' into master
kamalaboulhosn Jul 23, 2020
78ab82d
fix: Make error message more clear about where ordering must be enabl…
kamalaboulhosn Jul 23, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
4 changes: 3 additions & 1 deletion google-cloud-pubsub/src/main/java/com/google/cloud/pubsub/v1/Publisher.java
View file
Open in desktop
Expand Up @@ -233,7 +233,9 @@ public ApiFuture<String> publish(PubsubMessage message) {
final String orderingKey = message.getOrderingKey();
Preconditions.checkState(
orderingKey.isEmpty() || enableMessageOrdering,
"Cannot publish a message with an ordering key when message ordering is not enabled.");
"Cannot publish a message with an ordering key when message ordering is not enabled in the "
+ "Publisher client. Please create a Publisher client with "
+ "setEnableMessageOrdering(true) in the builder.");

final OutstandingPublish outstandingPublish =
new OutstandingPublish(messageTransform.apply(message));
Expand Down