Re: [PR] HBASE-27826 Add FSFT implementations for Virtual links and enable them as part of SplitProcedure [hbase]
apurtell merged PR #6936: URL: https://github.com/apache/hbase/pull/6936 -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-27826 Add FSFT implementations for Virtual links and enable them as part of SplitProcedure [hbase]
gvprathyusha6 commented on PR #6936: URL: https://github.com/apache/hbase/pull/6936#issuecomment-4310806814 @apurtell In the latest build, [large-wave-1 (pull_request)](https://github.com/apache/hbase/actions/runs/24854972507/job/72768089973?pr=6936) failed but only flakes failed and for [medium (pull_request)](https://github.com/apache/hbase/actions/runs/24854972507/job/72768090024?pr=6936) TestPrefetchWithBucketCache#testPrefetchRunTriggersEvictions has failed but it had passed in previous [runs](https://github.com/apache/hbase/actions/runs/24731401569/attempts/2) I also verified that TestPrefetchWithBucketCache#testPrefetchRunTriggersEvictions is successful with my patch on local. (Will create a separate jira to fix above) https://github.com/user-attachments/assets/0ba421f4-2c63-47aa-8fbc-82165eff8ad7"; /> -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-27826 Add FSFT implementations for Virtual links and enable them as part of SplitProcedure [hbase]
gvprathyusha6 commented on PR #6936: URL: https://github.com/apache/hbase/pull/6936#issuecomment-4302334482 > 4\. TestMultiVersions > There are a couple of issues here: > > 1. A possible Yetus version mismatch. Yetus complaining about unknown flags. > 2. Test failure: org.apache.hadoop.hbase.client.TestFromClientSideHTableExistsMethodMultipleRegionsMultipleGet > 3. Test failure: org.apache.hadoop.hbase.replication.TestSerialSyncReplication > 4. Test failure: org.apache.hadoop.hbase.TestMultiVersions > > I don't think these are related to this change. @gvprathyusha6 can you try the tests 3 and 4 with your change applied? @apurtell I ran tests 3 and 4 with my changes and they seem to be working, will push a commit amend with no diff and retrigger PR build once https://github.com/user-attachments/assets/0c37ad30-8b35-4155-9590-809e4d1ef0f5"; /> https://github.com/user-attachments/assets/0247b8fb-30d8-4ac5-b14d-6e16fae424aa"; /> -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-27826 Add FSFT implementations for Virtual links and enable them as part of SplitProcedure [hbase]
apurtell commented on PR #6936: URL: https://github.com/apache/hbase/pull/6936#issuecomment-4301078500 There are a couple of issues here: 1. A possible Yetus version mismatch. Yetus complaining about unknown flags. 2. Test failure: org.apache.hadoop.hbase.client.TestFromClientSideHTableExistsMethodMultipleRegionsMultipleGet 3. Test failure: org.apache.hadoop.hbase.replication.TestSerialSyncReplication 4. Test failure: org.apache.hadoop.hbase.TestMultiVersions I don't think these are related to this change. @gvprathyusha6 can you try the tests 3 and 4 with your change applied? -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-27826 Add FSFT implementations for Virtual links and enable them as part of SplitProcedure [hbase]
gvprathyusha6 commented on PR #6936: URL: https://github.com/apache/hbase/pull/6936#issuecomment-4255187373 > +1, lgtm Please address merge conflicts and I will look at this again. thanks @apurtell, uploaded patch to resolve conflicts -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
