[GitHub] [parquet-mr] shangxinli commented on pull request #918: PARQUET-2064: Make Range public accessible in RowRanges

2021-07-11 Thread GitBox
shangxinli commented on pull request #918: URL: https://github.com/apache/parquet-mr/pull/918#issuecomment-877865250 Thanks Chao for your comments. Can you make a PR for [PARQUET-2061](https://issues.apache.org/jira/browse/PARQUET-2061)? I am thinking we can add a new API to return the Row

[GitHub] [parquet-mr] shangxinli commented on pull request #918: PARQUET-2064: Make Range public accessible in RowRanges

2021-07-17 Thread GitBox
shangxinli commented on pull request #918: URL: https://github.com/apache/parquet-mr/pull/918#issuecomment-881969803 @gszadovszky I just made the change but the building error 'METHOD_RETURN_TYPE_CHANGED' was reported by 'japicmp'. I guess we need to change something to let 'japicmp' know

[GitHub] [parquet-mr] shangxinli commented on pull request #918: PARQUET-2064: Make Range public accessible in RowRanges

2021-08-02 Thread GitBox
shangxinli commented on pull request #918: URL: https://github.com/apache/parquet-mr/pull/918#issuecomment-891051718 Agree. Let's keep it internal for now. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

[GitHub] [parquet-mr] shangxinli commented on pull request #918: PARQUET-2064: Make Range public accessible in RowRanges

2021-08-04 Thread GitBox
shangxinli commented on pull request #918: URL: https://github.com/apache/parquet-mr/pull/918#issuecomment-892720333 Thank you so much @gszadovszky! Sorry didn't get time to check your ask yesterday. I see INFRA-22171 is moving forward. Let's see. -- This is an automated message from th

[GitHub] [parquet-mr] shangxinli commented on pull request #918: PARQUET-2064: Make Range public accessible in RowRanges

2021-08-05 Thread GitBox
shangxinli commented on pull request #918: URL: https://github.com/apache/parquet-mr/pull/918#issuecomment-892720333 Thank you so much @gszadovszky! Sorry didn't get time to check your ask yesterday. I see INFRA-22171 is moving forward. Let's see. -- This is an automated message from th