wchevreuil commented on PR #170: URL: https://github.com/apache/hbase-connectors/pull/170#issuecomment-6466973716
> Thanks for pointing to the earlier discussion. I understand that preserving the zero-length-to-null mapping in Spark 4 was a deliberate choice to match Spark 3, not an unexamined omission. PR #170 changes only the Spark 4 reader and would alter that established behavior, so I do not want to assume the design choice has changed. Would you prefer this PR be closed or reworked to preserve the existing mapping, or is a deliberate cross-version semantic change acceptable here? I have not changed the branch while clarifying this. I think we should keep the current implementation as it is, as it is consistent with the way spark3 module behaves and would be less impacting to client applications migrating from spark3 to spark4. -- 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]
