Code

Enable readColumn to read all rows#53

Code

Merged

platypii merged 15 commits intohyparam:masterhyparam/hyparquet:masterfrom park-brian:read-column-rowlimit-infinitepark-brian/hyparquet:read-column-rowlimit-infiniteCopy head branch name to clipboardDec 20, 2024

+94-7

Commits

Commits on Dec 14, 2024

• Enable readColumn to read all rows

  park-brian

  committed
  d6e04ea

• Refactor readColumn to use hasRowLimit

  park-brian

  committed
  b4a79b0

• Simplify hasRowLimit condition

  park-brian

  committed
  3a29c7d

• Check less common condition first

  park-brian

  committed
  24fa314

Commits on Dec 18, 2024

• Merge branch 'hyparam:master' into read-column-rowlimit-infinite

  park-brian

  authored
  ee45b24

• add readColumn test files

  park-brian

  committed
  764d40e

• implement readColumn tests for undefined rowLimits

  park-brian

  committed
  afb38b8

• remove unused variable

  park-brian

  committed
  4b25716

• return early if no metadata is present

  park-brian

  committed
  acb49f7

• address tsc warnings

  park-brian

  committed
  6da005b

• add comparison

  park-brian

  committed
  0426230

• clarify that undefined is valid for rowLimit

  park-brian

  committed
  dab11c2

Commits on Dec 20, 2024

• remove test files

  park-brian

  committed
  481c5d7

• verify edge case works when rowLimit is undefined

  park-brian

  committed
  3eb6568

• add test cases for readColumn

  park-brian

  committed
  79ccbe3