Skip to content

Commit d101c30

Browse files
tonyjongyoonanlarry-safran
authored andcommitted
Context.Storage deprecated method attach() removed, doAttach() made abstract (grpc#10379)
1 parent 67daa50 commit d101c30

File tree

1 file changed

+1
-17
lines changed

1 file changed

+1
-17
lines changed

api/src/context/java/io/grpc/Context.java

Lines changed: 1 addition & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -999,16 +999,6 @@ public String toString() {
999999
* subject to change.
10001000
*/
10011001
public abstract static class Storage {
1002-
/**
1003-
* Unused.
1004-
*
1005-
* @deprecated This is an old API that is no longer used.
1006-
*/
1007-
@Deprecated
1008-
public void attach(Context toAttach) {
1009-
throw new UnsupportedOperationException("Deprecated. Do not call.");
1010-
}
1011-
10121002
/**
10131003
* Implements {@link io.grpc.Context#attach}.
10141004
*
@@ -1022,13 +1012,7 @@ public void attach(Context toAttach) {
10221012
* as the {@code toRestore} parameter. {@code null} is a valid return value, but see
10231013
* caution note.
10241014
*/
1025-
public Context doAttach(Context toAttach) {
1026-
// This is a default implementation to help migrate existing Storage implementations that
1027-
// have an attach() method but no doAttach() method.
1028-
Context current = current();
1029-
attach(toAttach);
1030-
return current;
1031-
}
1015+
public abstract Context doAttach(Context toAttach);
10321016

10331017
/**
10341018
* Implements {@link io.grpc.Context#detach}.

0 commit comments

Comments
 (0)