-
Notifications
You must be signed in to change notification settings - Fork 3.3k
HBASE-29268: Restoring to original splits will fail for remote file systems not on the RS classpath #6915
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
… the RS classpath
This should be covered by existing unit tests, as the underlying behavior shouldn't have changed |
} | ||
}); | ||
@Override | ||
protected Path createInputFileListing(Job job) throws IOException { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to do the same thing we're currently doing here or else the subsequent bulkload doesn't work correctly, specifically for incremental backups
* See the License for the specific language governing permissions and | ||
* limitations under the License. | ||
*/ | ||
package org.apache.hadoop.hbase.backup.mapreduce; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is all c/p from MapReduceBackupCopyJob so we can share the logic
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
cc @rmdmattingly @ndimiduk