Currently, many modules depend on `managed-ledger-test.jar` just because they want to use MockBookkeeper and MockZookeeper.  This made module dependencies hard to understand and track.

This PR introduces a new `testmocks` module and pulls all mocks from managed-ledger tests into the new module.

### Motivation
The removed test libraries were already defined in the parent pom

### Modification
Removed duplicated test libraries in POM dependencies

* Switch ManagedLedger to use MetadataStore interface

* Use different thread for ZK callbacks

* Properly close metadata store in ml factory

* Fixed licenses

* Fixed problem in MockZookeeper

*Motivation*

Bump the development version to 2.6.0-SNAPSHOT

* Removed managed-ledger-shaded artifact

* Fixed dep artifact renaming error

* adding metrics to presto pulsar connector

* rename batch size

* adding comments

* refactoring metrics

* modifying bytes read metric

* fixing tests

* deleting tmp file

* adding jars to LICENSE

* adding a module that shades managed-ledger

* removing commented out lines

Signed-off-by: NSijie Guo <sijie@apache.org>

* Removed internal shading

* Fixed handling of IllegalArgumentException in ZK client wrapper

* Fixed tests with real ZK server

* Used shaded version of protobuf-2.4.1 internally, to coexist with components using proto3

* 2.1 parent pom

* Add proto3 dependency back since before it was relying on the proto2 dependency

* Fixed dependency

Revert "Fix for cursorPersistenceAsyncMarkDeleteSameThread (#1544)"
This reverts commit ede7ffe5.

Revert "Fixed flaky test in managed ledger close path (#1539)"
This reverts commit b22769bb.

Revert "Avoid contention in ManagedCursorImpl generated by locking on pendingMarkDeleteOps (#1519)"
This reverts commit ed8d1a5d.

* Enforce timeout on all TestNG tests, thread dump on timeout

* Increased default timeout to 120 seconds

* Removed usage of log statement that were making the log4j appender test to fail

* Move pulsar functions dependency version to root pom and remove duplicated license headers

This addresses some comments in pulsar functions PR #1314

* shade worker

* Fix broken master

* Upgrade the bookkeeper storage client dependency to the official bookkeeper version

This removes the temp dependency in `pulsar-functions-instance`

* set `protobuf2.version` in pulsar-common

* provide a shaded worker

* include worker dependency at broker

* Embeded function worker at broker

* rename 'function worker' to 'functions worker'

* add "--no-functions-worker" for pulsar-client-cpp tests

* Switched to BookKeeper-4.7.0-SNAPSHOT + Adjustements

* Added some missing operations to GrowableArrayBlockingQueue

Managed ledger was using UnboundedArrayBlockingQueue from yahoo/bookkeeper branch.

* Removed mock BK classes that are now in BK itself

* Fixed flaky test after refactoring

* Fixed some other tests

* One more flaky test

* Added bk tests dependency to pulsar-client-tools

* Introduce retries for flaky tests

* Avoid conflicts in allocating ports across different processes

* Removed duplicated builtools module after merge

* Also do retries on pulsar-discovery-service

Move zookeeper/bookkeeper from dependency management and only specify the dependency when they are used (#1052)

* Updates headers with ASF source files header

* change license to additional files