arrow

MINOR: [Java] Fix dependencies scope in flight-sql module (#36132)

### Rationale for this change
flight-sql module advertises dependencies which are not actually used by the published code, causing those dependencies to show up in downstream projects.

### What changes are included in this PR?
Fix dependencies scope in flight-sql module:
* move hamcrest to test scope as it is only used during unit tests
* add optional flag for commons-cli as it is only used in the example class

### Are these changes tested?
No new test. Confirming that there's no build issue

### Are there any user-facing changes?
No

Authored-by: Laurent Goujon <laurent@apache.org>
Signed-off-by: David Li <li.davidm96@gmail.com>