gjacoby126 commented on PR #1488: URL: https://github.com/apache/phoenix/pull/1488#issuecomment-1215502759
@gurpreet0909 - I believe this patch would break MapReduce jobs that do not have isNamespaceMappingEnabled set to true. (You can see, for example, that in the Jenkins test run that TableSnapshotReadsMapReduceIT has several failures for this reason.) Shouldn't we only set the MapReduce client to isNamespaceMappingEnabled = true if the cluster is in fact namespace enabled? -- 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]
