The checksyle plugin was added to the pulsar sql module to enforce the defined style. All violations were fixed:

- Ordering of imports.
- Formatting of the code.
- Absent Javadoc comments.
- Other small issues.
(cherry picked from commit f6fee1c6)

### Motivation

Fix #4732

### Modifications

Add options to rewrite the namespace delimiter, disable by default

Enable rewrite namespace delimiter can work well with superset:
<img width="1279" alt="superset" src="https://user-images.githubusercontent.com/12592133/61385412-f0f35700-a8e4-11e9-87b2-a31b62128b58.png">

### Does this pull request potentially affect one of the following parts:

*If `yes` was chosen, please highlight the changes*

  - Dependencies (does it add or upgrade a dependency): (no)
  - The public API: (no)
  - The schema: (no)
  - The default values of configurations: (no)
  - The wire protocol: (no)
  - The rest endpoints: (no)
  - The admin cli options: (no)
  - Anything that affects deployment: (no)

### Documentation

  - Does this pull request introduce a new feature? (no)

(cherry picked from commit 6ddd51ff)

* Adding offloader support for sql

* cleaning up

* cleaning up imports

* cleaning up configs

* fix imports

* fix behavior when offloader not configured and fix license

* fix unit test

* Get information_schema for pulsar connector to work

* fix unit test

* 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

### Motivation

Improve error message handling in presto pulsar connector

* adding module for Pulsar SQL

* renaming presto pulsar package

* fixing pom and configs

* fixing pom

* using project version variable in pom

* adding comments

* Updates headers with ASF source files header

* change license to additional files

* Introduce new load manager API

* Fix formatting and style issues

* Fix imports

* Add back renamed classes

* Add policies to ModularLoadManagerImpl

* Make system resource usage more influential

* Fix reflective setting of load manager in tests

* Conform with mockito syntax

* Improve formatting and design

* Revert to old simulation behavior, use reflective interface design

* Remove unnecessary async call

* Fix formatting

* Remove git merge remnant

* Refactor some classes and put loadManagerClassName in dynamic conf

* Delete LoadSimulationServer

* Clean API, share more code

* Remove redundant synchronized block, make correction to onUpdate

* Fix policy filter bug

* Remove availableActiveBrokers.get() call

* Use connection pooling

* Remove unused code

* Don't try reading the symlink and converting to absolute path

* Remove redundant and unnecessary statments

* Add test back to see if path contains /virtual/

* Replace System.out statements with log.info

* Change %s to {}