Verify each finding against the current code and only fix it if needed.

Inline comments:
In
`@core/src/main/java/io/questdb/griffin/engine/table/ConcurrentTimeFrameCursorImpl.java`:
- Around line 49-67: Reorder the private instance fields in class
ConcurrentTimeFrameCursorImpl so they are alphabetized by name while keeping
their modifiers, initializers and associated comments intact; specifically
arrange frameCount, frameCursor, frameMemoryPool, framePartitionIndexes,
frameRowCounts, frameTimestampCache, partitionCeilings, partitionTimestamps,
record, timeFrame, and timestampIndex into alphabetical order within the private
instance group and preserve the "// Cursor's lifecycle..." and timestamp-index
comment blocks adjacent to their respective fields.

In
`@core/src/main/java/io/questdb/griffin/engine/table/SelectedRecordCursorFactory.java`:
- Around line 188-195: In SelectedRecordCursorFactory, relocate the public
instance method newTimeFrameCursor() so public methods remain alphabetically
ordered: move the newTimeFrameCursor() method to sit between isProjection() and
recordCursorSupportsLongTopK(); keep the method body unchanged (it constructs a
SelectedConcurrentTimeFrameCursor when crossedIndex is true) and ensure
imports/visibility remain the same.

In `@core/src/test/java/io/questdb/test/griffin/HorizonJoinTest.java`:
- Around line 639-671: Update the SQL literals passed into the
assertQueryNoLeakCheck calls so keywords use uppercase IN and the SQL is written
as a Java text block (multiline string) instead of concatenated string
expressions; specifically modify the two queries used in the HorizonJoinTest's
assertQueryNoLeakCheck calls (the HORIZON JOIN query and the variant with
"SELECT price, price price0, sym, ts FROM prices WHERE ts in '1970-01-01'") to
replace "in" with "IN" and convert the concatenated/quoted SQL to a single
"""...""" text block each, keeping the same expressions for getSecondsDivisor()
and the surrounding Java code (these changes affect the SQL literals used by
SelectedConcurrentTimeFrameCursor test paths).

In `@core/src/test/java/io/questdb/test/griffin/WindowJoinTest.java`:
- Around line 3390-3448: The test method
testWindowJoinSelfJoinWithAggregatesInSelectAndWhere uses long concatenated
string literals for the CREATE/INSERT and the SELECT+expected result; change
these to Java multiline text blocks ("""...""") to improve readability and match
test SQL formatting rules—replace the concatenated execute("INSERT INTO
fx_trades VALUES " + "...") call and the long "SELECT " + "t.timestamp, " + ...
string passed into assertQueryNoLeakCheck, as well as the expected result block,
with proper text blocks while keeping calls to execute(...) and
assertQueryNoLeakCheck(...) and their argument order unchanged.
- Around line 3450-3527: Update the testWindowJoinSelfJoinWithVwapCalculation
test to follow formatting rules: replace large numeric literals 10000 and
2000000 with underscore-separated forms (e.g., 10_000, 2_000_000) in the INSERT
expression and any setProperty values if applicable, and convert the long SQL
string assignments (the "insert into fx_trades ..." and the failing query
assigned to query) to Java text blocks (multiline string literals) to improve
readability; edit the string variables used in engine.execute(...) and the query
variable inside testWindowJoinSelfJoinWithVwapCalculation and keep all semantics
unchanged.

---

Outside diff comments:
In
`@core/src/main/java/io/questdb/griffin/engine/table/SelectedRecordCursorFactory.java`:
- Around line 135-144: The getPageFrameCursor method assumes
base.getPageFrameCursor() returns a TablePageFrameCursor and casts
unconditionally; add a defensive instanceof check before casting and calling
SelectedPageFrameCursor.wrap so non-table-backed cursors (e.g.,
ReadParquetPageFrameCursor, RegisteredPageFrameCursor) are returned unchanged.
Specifically, in SelectedRecordCursorFactory.getPageFrameCursor, verify
baseCursor instanceof TablePageFrameCursor before calling
pageFrameCursor.wrap((TablePageFrameCursor) baseCursor) and return baseCursor if
not; apply the same pattern where SelectedPageFrameCursor.of/.wrap is used
(including the PartitionFrameCursor initialization path) to avoid
ClassCastException and the UnsupportedOperationException from incorrect
initialization.