ianawilson commented on pull request #88:
URL: https://github.com/apache/hbase-connectors/pull/88#issuecomment-1002313179


   One thing to think about with this:
   
   HBase 2.4.1 introduced (via this PR 
https://github.com/apache/hbase/pull/2800) a breaking change for 
`bulkLoadThinRows()` by moving some of the static helpers from `HStore` to 
`StoreUtils` (this was precisely the problem that lead me to this PR).
   
   This PR enables you to use this with HBase 2.4.1 and later, but will prevent 
you from using it with earlier HBase versions. I know you currently allow the 
HBase version to be configured via the system properties, but as-is this change 
won't work with the HBase library prior to 2.4.1.


-- 
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]


Reply via email to