[GitHub] [carbondata] akashrn5 commented on a change in pull request #3770: [CARBONDATA-3829] Support pagination in SDK reader

2020-06-03 Thread GitBox
akashrn5 commented on a change in pull request #3770: URL: https://github.com/apache/carbondata/pull/3770#discussion_r434515732 ## File path: sdk/sdk/src/main/java/org/apache/carbondata/sdk/file/PaginationCarbonReader.java ## @@ -0,0 +1,302 @@ +/* + * Licensed to the Apache So

[GitHub] [carbondata] akashrn5 commented on a change in pull request #3770: [CARBONDATA-3829] Support pagination in SDK reader

2020-06-03 Thread GitBox
akashrn5 commented on a change in pull request #3770: URL: https://github.com/apache/carbondata/pull/3770#discussion_r434515638 ## File path: sdk/sdk/src/main/java/org/apache/carbondata/sdk/file/PaginationCarbonReader.java ## @@ -0,0 +1,302 @@ +/* + * Licensed to the Apache So

[GitHub] [carbondata] akashrn5 commented on a change in pull request #3770: [CARBONDATA-3829] Support pagination in SDK reader

2020-06-03 Thread GitBox
akashrn5 commented on a change in pull request #3770: URL: https://github.com/apache/carbondata/pull/3770#discussion_r434515260 ## File path: sdk/sdk/src/main/java/org/apache/carbondata/sdk/file/PaginationCarbonReader.java ## @@ -0,0 +1,302 @@ +/* + * Licensed to the Apache So

[GitHub] [carbondata] akashrn5 commented on a change in pull request #3770: [CARBONDATA-3829] Support pagination in SDK reader

2020-06-03 Thread GitBox
akashrn5 commented on a change in pull request #3770: URL: https://github.com/apache/carbondata/pull/3770#discussion_r434379315 ## File path: sdk/sdk/src/test/java/org/apache/carbondata/sdk/file/PaginationCarbonReaderTest.java ## @@ -0,0 +1,221 @@ +/* + * Licensed to the Apach

[GitHub] [carbondata] akashrn5 commented on a change in pull request #3770: [CARBONDATA-3829] Support pagination in SDK reader

2020-06-03 Thread GitBox
akashrn5 commented on a change in pull request #3770: URL: https://github.com/apache/carbondata/pull/3770#discussion_r434325061 ## File path: docs/configuration-parameters.md ## @@ -141,6 +141,7 @@ This section provides the details of all the configurations required for the Ca