Remove a couple unused test methods
Add a test that demonstrates the problem.
Make all scans capable of visiting their operands, not just IndexScan.
merge MMcM: Make all scans capable of visiting their operands, not just IndexScan

Merge pull request #1504 from MMcM/sql-layer/sql-visit-non-index-scan-conds
https://github.com/FoundationDB/sql-layer/pull/1504
Switch from NewRow to Row in ApiTestBase/ITs
Rename to expectRows
Stop using ScanRequest inside ApiTestBase
Updated tests
Minor test fixes
Stale comment
merge SytzeH: Updated name for fdb-sql referring to fts test

Merge pull request #1505 from SytzeH/sql-layer/fdb-sql2
https://github.com/FoundationDB/sql-layer/pull/1505
merge nathanlws: Switch tests from NewRow to Row

Merge pull request #1506 from nathanlws/sql-layer/apitestbase-newrow-to-row
https://github.com/FoundationDB/sql-layer/pull/1506
Merge remote-tracking branch 'github/master' into Implement-new-HKey

Conflicts:
	src/test/java/com/foundationdb/server/test/it/qp/FlattenIT.java