Re: Dictionary Key For Null Slot

2022-11-30 Thread Raphael Taylor-Davies
For those following along, I've proposed a workaround that loosens the restriction on non-nullable children here . In particular non-nullable children are now allowed to contain nulls, so long as they don't introduce any new nulls not already found

Current state of using GitHub issues for Arrow

2022-11-30 Thread Joris Van den Bossche
Hi all, There is a separate vote thread about already using GitHub issues for all new issues (and in practice users also are already doing that, since new JIRA signup has been disabled). And so to prepare for this, there has been some ongoing work to update our workflows to handle GitHub issues (b

Re: DISCUSS: [FlightSQL] Catalog support

2022-11-30 Thread James Duong
Our current convention of sending connection properties as headers with every request has the benefit of making statefulness optional, but has the drawback of sending redundant, unused properties on requests after the first, which increases the payload size unnecessarily. I'd suggest we define ses

Re: DISCUSS: [FlightSQL] Catalog support

2022-11-30 Thread Gavin Ray
Just to chime in on this, one thing I'm curious about is whether there will be support for user-defined catalog/schema hierarchy depth? This comment that James made does seem reasonable to me > scheme://:/path-1/path-2/.../path-n Trino/Presto does a similar thing (jdbc:trino://localhost:8080/tpch

Re: [DISCUSS] JSON Canonical Extension Type

2022-11-30 Thread Micah Kornfield
> > Can a logical extension be based on another logical extension? Potentially but this is mostly an implementation details, each type should have their own specification IMO. HOCON support might be nice.. I'm not sure if this is common enough to warrant a canonical type within Arrow but you are

Re: Array::GetValue ?

2022-11-30 Thread Micah Kornfield
Yeah, another base class doesn't seem great. One option would be to use a union type like variant, but that seems also not particularly great? On Thu, Nov 17, 2022 at 1:15 AM Antoine Pitrou wrote: > > Uh, you're right. We may want another base class, not sure how it > should be named though

Re: Array::GetValue ?

2022-11-30 Thread John Muehlhausen
std::pair FlatArray::GetValueBytes(int64_t index) .second being -1 in every case other than a boolean ... in that case it indicates the bit of importance in the single-byte string_view? Just an idea. On Wed, Nov 30, 2022 at 11:46 PM Micah Kornfield wrote: > Yeah, another base class doesn't see