vertx-4-migration-guide icon indicating copy to clipboard operation
vertx-4-migration-guide copied to clipboard

Add note about removing Handler<AsyncResult<T>> from Future to migration guide.

Open LuboViluda opened this issue 4 years ago • 0 comments

Describe the feature

  • Add a note about the change to the migration guide.
  • Remove misleading part: "Future<T> extends Handler<AsyncResult<T>> for some time now (and the default implementation actually returns this)." source, which is no longer valid as Future doesn't extend Handler<AsyncResult<T> now.

Vert.x Future extends Handler<AsyncResult<T>>
Vert.x 3.9

Vert.x Future doesn't extend Handler<AsyncResult<T>> in Vert.x 4.0 Vert.x 4.0

As the handle(AsyncResult<T> asyncResult) method is missing this is a potentially breaking change.

Use cases

  • users of Vert.x are prompted about potentially breaking change.

LuboViluda avatar Jan 02 '21 15:01 LuboViluda