You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2020/12/15 11:00:48 UTC

[GitHub] [arrow] mqy opened a new pull request #8924: ARROW-10915: [Rust] README.md: set the Env vars as absolute dirs; several minor fixes.

mqy opened a new pull request #8924:
URL: https://github.com/apache/arrow/pull/8924


   In rust/README.md,  both ARROW_TEST_DATA and PARQUET_TEST_DATA are set as relative path. The problem is: we MAY have to reset them back-and-forth across top and subdirectories – that's annoying. So, the obvious solution is: set the Env vars as absolute dirs.
   
   This PR also contains several minor fixes to spelling and format!


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [arrow] codecov-io commented on pull request #8924: ARROW-10915: [Rust] README.md: set the Env vars as absolute dirs; several minor fixes.

Posted by GitBox <gi...@apache.org>.
codecov-io commented on pull request #8924:
URL: https://github.com/apache/arrow/pull/8924#issuecomment-745222383


   # [Codecov](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=h1) Report
   > Merging [#8924](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=desc) (8f3d8b6) into [master](https://codecov.io/gh/apache/arrow/commit/8df91c93676f8ac2b4d072e8ef0df62d3430cd57?el=desc) (8df91c9) will **increase** coverage by `23.79%`.
   > The diff coverage is `40.00%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/arrow/pull/8924/graphs/tree.svg?width=650&height=150&src=pr&token=LpTCFbqVT1)](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=tree)
   
   ```diff
   @@             Coverage Diff             @@
   ##           master    #8924       +/-   ##
   ===========================================
   + Coverage   51.89%   75.69%   +23.79%     
   ===========================================
     Files         173      181        +8     
     Lines       31081    41012     +9931     
   ===========================================
   + Hits        16131    31042    +14911     
   + Misses      14950     9970     -4980     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [rust/arrow/src/array/equal/mod.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9hcnJvdy9zcmMvYXJyYXkvZXF1YWwvbW9kLnJz) | `91.91% <ø> (+0.37%)` | :arrow_up: |
   | [rust/arrow/src/datatypes.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9hcnJvdy9zcmMvZGF0YXR5cGVzLnJz) | `76.83% <ø> (ø)` | |
   | [rust/arrow/src/error.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9hcnJvdy9zcmMvZXJyb3IucnM=) | `10.00% <0.00%> (ø)` | |
   | [rust/arrow/src/ffi.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9hcnJvdy9zcmMvZmZpLnJz) | `70.28% <0.00%> (-0.34%)` | :arrow_down: |
   | [rust/arrow/src/compute/kernels/substring.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9hcnJvdy9zcmMvY29tcHV0ZS9rZXJuZWxzL3N1YnN0cmluZy5ycw==) | `98.29% <100.00%> (ø)` | |
   | [rust/parquet/src/bin/parquet-schema.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9wYXJxdWV0L3NyYy9iaW4vcGFycXVldC1zY2hlbWEucnM=) | `0.00% <0.00%> (ø)` | |
   | [rust/parquet/src/util/test\_common/file\_util.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9wYXJxdWV0L3NyYy91dGlsL3Rlc3RfY29tbW9uL2ZpbGVfdXRpbC5ycw==) | `77.77% <0.00%> (ø)` | |
   | [rust/parquet/src/util/bit\_packing.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9wYXJxdWV0L3NyYy91dGlsL2JpdF9wYWNraW5nLnJz) | `99.96% <0.00%> (ø)` | |
   | [rust/parquet/src/bin/parquet-rowcount.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9wYXJxdWV0L3NyYy9iaW4vcGFycXVldC1yb3djb3VudC5ycw==) | `0.00% <0.00%> (ø)` | |
   | [rust/parquet/src/file/footer.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9wYXJxdWV0L3NyYy9maWxlL2Zvb3Rlci5ycw==) | `96.22% <0.00%> (ø)` | |
   | ... and [53 more](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=footer). Last update [8d76312...8f3d8b6](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [arrow] github-actions[bot] commented on pull request #8924: ARROW-10915: [Rust] README.md: set the Env vars as absolute dirs; several minor fixes.

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #8924:
URL: https://github.com/apache/arrow/pull/8924#issuecomment-745222218


   https://issues.apache.org/jira/browse/ARROW-10915


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [arrow] jorgecarleitao commented on pull request #8924: ARROW-10915: [Rust] README.md: set the Env vars as absolute dirs; several minor fixes.

Posted by GitBox <gi...@apache.org>.
jorgecarleitao commented on pull request #8924:
URL: https://github.com/apache/arrow/pull/8924#issuecomment-745254227


   Doesn't this assume the existence of `realpath`? I can't find that on mac os, and I am not sure it exists on windows also.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [arrow] mqy commented on pull request #8924: ARROW-10915: [Rust] README.md: set the Env vars as absolute dirs; several minor fixes.

Posted by GitBox <gi...@apache.org>.
mqy commented on pull request #8924:
URL: https://github.com/apache/arrow/pull/8924#issuecomment-745269927


   > Doesn't this assume the existence of `realpath`? I can't find that on mac os, and I am not sure it exists on windows also.
   
   Thanks, that's make sense. The bash on my macOS must have been replace by GNU bash 4. 
   I'll fix them with $(cd ...; pwd)


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [arrow] codecov-io edited a comment on pull request #8924: ARROW-10915: [Rust] README.md: set the Env vars as absolute dirs; several minor fixes.

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on pull request #8924:
URL: https://github.com/apache/arrow/pull/8924#issuecomment-745222383


   # [Codecov](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=h1) Report
   > Merging [#8924](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=desc) (3e5c84b) into [master](https://codecov.io/gh/apache/arrow/commit/8df91c93676f8ac2b4d072e8ef0df62d3430cd57?el=desc) (8df91c9) will **increase** coverage by `23.78%`.
   > The diff coverage is `40.00%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/arrow/pull/8924/graphs/tree.svg?width=650&height=150&src=pr&token=LpTCFbqVT1)](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=tree)
   
   ```diff
   @@             Coverage Diff             @@
   ##           master    #8924       +/-   ##
   ===========================================
   + Coverage   51.89%   75.68%   +23.78%     
   ===========================================
     Files         173      181        +8     
     Lines       31081    41012     +9931     
   ===========================================
   + Hits        16131    31041    +14910     
   + Misses      14950     9971     -4979     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [rust/arrow/src/array/equal/mod.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9hcnJvdy9zcmMvYXJyYXkvZXF1YWwvbW9kLnJz) | `91.91% <ø> (+0.37%)` | :arrow_up: |
   | [rust/arrow/src/datatypes.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9hcnJvdy9zcmMvZGF0YXR5cGVzLnJz) | `76.83% <ø> (ø)` | |
   | [rust/arrow/src/error.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9hcnJvdy9zcmMvZXJyb3IucnM=) | `10.00% <0.00%> (ø)` | |
   | [rust/arrow/src/ffi.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9hcnJvdy9zcmMvZmZpLnJz) | `70.28% <0.00%> (-0.34%)` | :arrow_down: |
   | [rust/arrow/src/compute/kernels/substring.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9hcnJvdy9zcmMvY29tcHV0ZS9rZXJuZWxzL3N1YnN0cmluZy5ycw==) | `98.29% <100.00%> (ø)` | |
   | [rust/parquet\_derive/src/lib.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9wYXJxdWV0X2Rlcml2ZS9zcmMvbGliLnJz) | `0.00% <0.00%> (ø)` | |
   | [rust/parquet/src/util/hash\_util.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9wYXJxdWV0L3NyYy91dGlsL2hhc2hfdXRpbC5ycw==) | `95.77% <0.00%> (ø)` | |
   | [rust/parquet/src/util/bit\_packing.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9wYXJxdWV0L3NyYy91dGlsL2JpdF9wYWNraW5nLnJz) | `99.96% <0.00%> (ø)` | |
   | [rust/parquet/src/file/footer.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9wYXJxdWV0L3NyYy9maWxlL2Zvb3Rlci5ycw==) | `96.22% <0.00%> (ø)` | |
   | [rust/parquet/src/bin/parquet-rowcount.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9wYXJxdWV0L3NyYy9iaW4vcGFycXVldC1yb3djb3VudC5ycw==) | `0.00% <0.00%> (ø)` | |
   | ... and [53 more](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=footer). Last update [8d76312...3e5c84b](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [arrow] mqy edited a comment on pull request #8924: ARROW-10915: [Rust] README.md: set the Env vars as absolute dirs; several minor fixes.

Posted by GitBox <gi...@apache.org>.
mqy edited a comment on pull request #8924:
URL: https://github.com/apache/arrow/pull/8924#issuecomment-745269927


   > Doesn't this assume the existence of `realpath`? I can't find that on mac os, and I am not sure it exists on windows also.
   
   Thanks, that's make sense. The bash on my macOS must have been replaced by GNU bash 5. 
   I'll fix them with $(cd ...; pwd)


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [arrow] codecov-io edited a comment on pull request #8924: ARROW-10915: [Rust] README.md: set the Env vars as absolute dirs; several minor fixes.

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on pull request #8924:
URL: https://github.com/apache/arrow/pull/8924#issuecomment-745222383


   # [Codecov](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=h1) Report
   > Merging [#8924](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=desc) (23f8b47) into [master](https://codecov.io/gh/apache/arrow/commit/8df91c93676f8ac2b4d072e8ef0df62d3430cd57?el=desc) (8df91c9) will **increase** coverage by `23.45%`.
   > The diff coverage is `40.00%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/arrow/pull/8924/graphs/tree.svg?width=650&height=150&src=pr&token=LpTCFbqVT1)](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=tree)
   
   ```diff
   @@             Coverage Diff             @@
   ##           master    #8924       +/-   ##
   ===========================================
   + Coverage   51.89%   75.35%   +23.45%     
   ===========================================
     Files         173      177        +4     
     Lines       31081    40822     +9741     
   ===========================================
   + Hits        16131    30763    +14632     
   + Misses      14950    10059     -4891     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [rust/arrow/src/array/equal/mod.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9hcnJvdy9zcmMvYXJyYXkvZXF1YWwvbW9kLnJz) | `91.91% <ø> (+0.37%)` | :arrow_up: |
   | [rust/arrow/src/datatypes.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9hcnJvdy9zcmMvZGF0YXR5cGVzLnJz) | `76.83% <ø> (ø)` | |
   | [rust/arrow/src/error.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9hcnJvdy9zcmMvZXJyb3IucnM=) | `10.00% <0.00%> (ø)` | |
   | [rust/arrow/src/ffi.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9hcnJvdy9zcmMvZmZpLnJz) | `70.28% <0.00%> (-0.34%)` | :arrow_down: |
   | [rust/arrow/src/compute/kernels/substring.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9hcnJvdy9zcmMvY29tcHV0ZS9rZXJuZWxzL3N1YnN0cmluZy5ycw==) | `98.29% <100.00%> (ø)` | |
   | [rust/parquet/src/file/footer.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9wYXJxdWV0L3NyYy9maWxlL2Zvb3Rlci5ycw==) | `96.22% <0.00%> (ø)` | |
   | [rust/parquet/src/util/bit\_packing.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9wYXJxdWV0L3NyYy91dGlsL2JpdF9wYWNraW5nLnJz) | `99.96% <0.00%> (ø)` | |
   | [rust/parquet/src/util/hash\_util.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9wYXJxdWV0L3NyYy91dGlsL2hhc2hfdXRpbC5ycw==) | `95.77% <0.00%> (ø)` | |
   | [rust/parquet/src/util/test\_common/file\_util.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9wYXJxdWV0L3NyYy91dGlsL3Rlc3RfY29tbW9uL2ZpbGVfdXRpbC5ycw==) | `77.77% <0.00%> (ø)` | |
   | [rust/arrow/src/compute/kernels/cast.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9hcnJvdy9zcmMvY29tcHV0ZS9rZXJuZWxzL2Nhc3QucnM=) | `96.33% <0.00%> (+0.12%)` | :arrow_up: |
   | ... and [47 more](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=footer). Last update [0e8de08...9ab561f](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [arrow] mqy edited a comment on pull request #8924: ARROW-10915: [Rust] README.md: set the Env vars as absolute dirs; several minor fixes.

Posted by GitBox <gi...@apache.org>.
mqy edited a comment on pull request #8924:
URL: https://github.com/apache/arrow/pull/8924#issuecomment-745269927


   > Doesn't this assume the existence of `realpath`? I can't find that on mac os, and I am not sure it exists on windows also.
   
   Thanks, that's make sense. The bash on my macOS must have been replaced by GNU bash 4. 
   I'll fix them with $(cd ...; pwd)


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [arrow] codecov-io edited a comment on pull request #8924: ARROW-10915: [Rust] README.md: set the Env vars as absolute dirs; several minor fixes.

Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on pull request #8924:
URL: https://github.com/apache/arrow/pull/8924#issuecomment-745222383


   # [Codecov](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=h1) Report
   > Merging [#8924](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=desc) (db72711) into [master](https://codecov.io/gh/apache/arrow/commit/8df91c93676f8ac2b4d072e8ef0df62d3430cd57?el=desc) (8df91c9) will **increase** coverage by `23.79%`.
   > The diff coverage is `40.00%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/arrow/pull/8924/graphs/tree.svg?width=650&height=150&src=pr&token=LpTCFbqVT1)](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=tree)
   
   ```diff
   @@             Coverage Diff             @@
   ##           master    #8924       +/-   ##
   ===========================================
   + Coverage   51.89%   75.69%   +23.79%     
   ===========================================
     Files         173      181        +8     
     Lines       31081    41012     +9931     
   ===========================================
   + Hits        16131    31042    +14911     
   + Misses      14950     9970     -4980     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [rust/arrow/src/array/equal/mod.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9hcnJvdy9zcmMvYXJyYXkvZXF1YWwvbW9kLnJz) | `91.91% <ø> (+0.37%)` | :arrow_up: |
   | [rust/arrow/src/datatypes.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9hcnJvdy9zcmMvZGF0YXR5cGVzLnJz) | `76.83% <ø> (ø)` | |
   | [rust/arrow/src/error.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9hcnJvdy9zcmMvZXJyb3IucnM=) | `10.00% <0.00%> (ø)` | |
   | [rust/arrow/src/ffi.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9hcnJvdy9zcmMvZmZpLnJz) | `70.28% <0.00%> (-0.34%)` | :arrow_down: |
   | [rust/arrow/src/compute/kernels/substring.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9hcnJvdy9zcmMvY29tcHV0ZS9rZXJuZWxzL3N1YnN0cmluZy5ycw==) | `98.29% <100.00%> (ø)` | |
   | [rust/arrow/src/compute/kernels/comparison.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9hcnJvdy9zcmMvY29tcHV0ZS9rZXJuZWxzL2NvbXBhcmlzb24ucnM=) | `96.27% <0.00%> (ø)` | |
   | [rust/parquet/src/file/footer.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9wYXJxdWV0L3NyYy9maWxlL2Zvb3Rlci5ycw==) | `96.22% <0.00%> (ø)` | |
   | [rust/parquet/src/util/bit\_packing.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9wYXJxdWV0L3NyYy91dGlsL2JpdF9wYWNraW5nLnJz) | `99.96% <0.00%> (ø)` | |
   | [rust/parquet/src/bin/parquet-rowcount.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9wYXJxdWV0L3NyYy9iaW4vcGFycXVldC1yb3djb3VudC5ycw==) | `0.00% <0.00%> (ø)` | |
   | [rust/parquet\_derive/src/lib.rs](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree#diff-cnVzdC9wYXJxdWV0X2Rlcml2ZS9zcmMvbGliLnJz) | `0.00% <0.00%> (ø)` | |
   | ... and [54 more](https://codecov.io/gh/apache/arrow/pull/8924/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=footer). Last update [0e8de08...db72711](https://codecov.io/gh/apache/arrow/pull/8924?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [arrow] mqy commented on pull request #8924: ARROW-10915: [Rust] README.md: set the Env vars as absolute dirs; several minor fixes.

Posted by GitBox <gi...@apache.org>.
mqy commented on pull request #8924:
URL: https://github.com/apache/arrow/pull/8924#issuecomment-745280217


   Sorry, due to some strange problem (VsCode cache when do branch switch), I did some forced push.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [arrow] andygrove closed pull request #8924: ARROW-10915: [Rust] README.md: set the Env vars as absolute dirs; several minor fixes.

Posted by GitBox <gi...@apache.org>.
andygrove closed pull request #8924:
URL: https://github.com/apache/arrow/pull/8924


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org