Skip to content

Commit

Permalink
Modify comments according to reviewer's comment
Browse files Browse the repository at this point in the history
  • Loading branch information
kingcrimsontianyu committed Dec 18, 2024
1 parent e524647 commit d190146
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions cpp/src/io/orc/stripe_data.cu
Original file line number Diff line number Diff line change
Expand Up @@ -136,8 +136,8 @@ struct orcdec_state_s {
* @brief Manage caching of the first run of TIMESTAMP's DATA stream for a row group.
*
* This class is used to address a special case, where the first run spans two adjacent row groups
* and its length is greater than the maximum length allowed to be consumed (which limit is imposed
* by the decoder when processing the SECONDARY stream). This class shall be instantiated in the
* and its length is greater than the maximum length allowed to be consumed. This limit is imposed
* by the decoder when processing the SECONDARY stream. This class shall be instantiated in the
* shared memory. As an optimization, the actual cache is a local variable and does not reside in
* the shared memory.
*/
Expand Down
2 changes: 1 addition & 1 deletion python/cudf/cudf/tests/test_orc.py
Original file line number Diff line number Diff line change
Expand Up @@ -1984,7 +1984,7 @@ def test_orc_reader_desynced_timestamp(datadir, inputfile):
# is progressed faster than the SECONDARY stream (nanosecond) at the start of a row
# group. In this case, the "run cache manager" in the decoder kernel is used to
# orchestrate the dual-stream processing.
# See https://github.com/rapidsai/cudf/issues/17155 .
# For more information, see https://github.com/rapidsai/cudf/issues/17155.

path = datadir / inputfile

Expand Down

0 comments on commit d190146

Please sign in to comment.