[GitHub] [hbase] jojochuang commented on pull request #5189: HBASE-27746 Check if the file system supports storage policy before invoking setStoragePolicy()

2023-06-16 Thread via GitHub
jojochuang commented on PR #5189: URL: https://github.com/apache/hbase/pull/5189#issuecomment-159497 Well actually the test would fail if it's not "hdfs" `[ERROR] TestFSUtils.testSetStoragePolicyDefault:412->verifyNoHDFSApiInvocationForDefaultPolicy:422 ยป IllegalArgument

[GitHub] [hbase] jojochuang commented on pull request #5189: HBASE-27746 Check if the file system supports storage policy before invoking setStoragePolicy()

2023-04-20 Thread via GitHub
jojochuang commented on PR #5189: URL: https://github.com/apache/hbase/pull/5189#issuecomment-1516671554 The failed test doesn't look related. @taklwu can you help review this one? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [hbase] jojochuang commented on pull request #5189: HBASE-27746 Check if the file system supports storage policy before invoking setStoragePolicy()

2023-04-19 Thread via GitHub
jojochuang commented on PR #5189: URL: https://github.com/apache/hbase/pull/5189#issuecomment-1515407953 Thanks for the tips! Updated checkstyle and also fixed the test. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use