westonpace commented on a change in pull request #9323:
URL: https://github.com/apache/arrow/pull/9323#discussion_r580559082
##########
File path: cpp/src/arrow/dataset/partition_test.cc
##########
@@ -322,8 +397,9 @@ TEST_F(TestPartitioning, DiscoverHiveSchema) {
}
Review comment:
There was already one in dictionary inference but I added one in the
non-dictionary inference case too. This also prompted me to consider the case
`{"/alpha=_FALLBACK_STRING_"}` which I've changed to be an error instead of
returning an empty schema. This matches directory partitioning.
##########
File path: cpp/src/arrow/dataset/partition.h
##########
@@ -120,6 +125,7 @@ class ARROW_DS_EXPORT KeyValuePartitioning : public
Partitioning {
/// of a scalar value
struct Key {
std::string name, value;
+ bool null;
Review comment:
Done.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
