[jira] [Commented] (ARROW-5767) [Format] Permit dictionary replacements in IPC protocol

2020-01-08 Thread Micah Kornfield (Jira)


[ 
https://issues.apache.org/jira/browse/ARROW-5767?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17011038#comment-17011038
 ] 

Micah Kornfield commented on ARROW-5767:


yes, I think this duplicate ARROW-7283

> [Format] Permit dictionary replacements in IPC protocol
> ---
>
> Key: ARROW-5767
> URL: https://issues.apache.org/jira/browse/ARROW-5767
> Project: Apache Arrow
>  Issue Type: New Feature
>  Components: Format
>Reporter: Wes McKinney
>Priority: Major
> Fix For: 0.16.0
>
>
> We permit dictionaries to grow using the {{isDelta}} property in the IPC 
> protocol. I think it should be allowed for the same dictionary ID to appear 
> in an IPC protocol stream but with {{isDelta=false}}. This would indicate 
> that the dictionary in that message is to replace any prior-observed ones in 
> subsequent record batches.
> For example, we might have dictionary batches in a stream:
> {code}
> id: 0 isDelta: false values: [a, b, c]
> id: 0 isDelta: true values [d]
> id 0 isDelta: false values [c, a, b]
> {code}
> Such data could easily be produced by a stream producer that is creating 
> dictionaries in different execution threads. 



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (ARROW-5767) [Format] Permit dictionary replacements in IPC protocol

2020-01-07 Thread Wes McKinney (Jira)


[ 
https://issues.apache.org/jira/browse/ARROW-5767?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17010173#comment-17010173
 ] 

Wes McKinney commented on ARROW-5767:
-

I think this may be resolved now? [~emkornfield]

> [Format] Permit dictionary replacements in IPC protocol
> ---
>
> Key: ARROW-5767
> URL: https://issues.apache.org/jira/browse/ARROW-5767
> Project: Apache Arrow
>  Issue Type: New Feature
>  Components: Format
>Reporter: Wes McKinney
>Priority: Major
> Fix For: 0.16.0
>
>
> We permit dictionaries to grow using the {{isDelta}} property in the IPC 
> protocol. I think it should be allowed for the same dictionary ID to appear 
> in an IPC protocol stream but with {{isDelta=false}}. This would indicate 
> that the dictionary in that message is to replace any prior-observed ones in 
> subsequent record batches.
> For example, we might have dictionary batches in a stream:
> {code}
> id: 0 isDelta: false values: [a, b, c]
> id: 0 isDelta: true values [d]
> id 0 isDelta: false values [c, a, b]
> {code}
> Such data could easily be produced by a stream producer that is creating 
> dictionaries in different execution threads. 



--
This message was sent by Atlassian Jira
(v8.3.4#803005)