[GitHub] [hudi] TJX2014 commented on a diff in pull request #7220: [HUDI-5230] Lazy init secondaryView in PriorityBasedFileSystemView

2022-11-21 Thread GitBox
TJX2014 commented on code in PR #7220: URL: https://github.com/apache/hudi/pull/7220#discussion_r1028963797 ## hudi-common/src/test/java/org/apache/hudi/common/table/view/TestPriorityBasedFileSystemView.java: ## @@ -75,7 +75,8 @@ public class TestPriorityBasedFileSystemView {

[GitHub] [hudi] TJX2014 commented on a diff in pull request #7220: [HUDI-5230] Lazy init secondaryView in PriorityBasedFileSystemView

2022-11-20 Thread GitBox
TJX2014 commented on code in PR #7220: URL: https://github.com/apache/hudi/pull/7220#discussion_r1027560453 ## hudi-common/src/main/java/org/apache/hudi/common/table/view/PriorityBasedFileSystemView.java: ## @@ -131,149 +133,159 @@ private void

[GitHub] [hudi] TJX2014 commented on a diff in pull request #7220: [HUDI-5230] Lazy init secondaryView in PriorityBasedFileSystemView

2022-11-20 Thread GitBox
TJX2014 commented on code in PR #7220: URL: https://github.com/apache/hudi/pull/7220#discussion_r1027560371 ## hudi-common/src/main/java/org/apache/hudi/common/table/view/FileSystemViewManager.java: ## @@ -26,6 +26,7 @@ import

[GitHub] [hudi] TJX2014 commented on a diff in pull request #7220: [HUDI-5230] Lazy init secondaryView in PriorityBasedFileSystemView

2022-11-20 Thread GitBox
TJX2014 commented on code in PR #7220: URL: https://github.com/apache/hudi/pull/7220#discussion_r1027560298 ## hudi-common/src/main/java/org/apache/hudi/common/table/view/PriorityBasedFileSystemView.java: ## @@ -25,6 +25,7 @@ import

[GitHub] [hudi] TJX2014 commented on a diff in pull request #7220: [HUDI-5230] Lazy init secondaryView in PriorityBasedFileSystemView

2022-11-20 Thread GitBox
TJX2014 commented on code in PR #7220: URL: https://github.com/apache/hudi/pull/7220#discussion_r1027556918 ## hudi-common/src/test/java/org/apache/hudi/common/table/view/TestPriorityBasedFileSystemView.java: ## @@ -75,7 +75,8 @@ public class TestPriorityBasedFileSystemView {