chaoyli commented on pull request #586:
URL: https://github.com/apache/orc/pull/586#issuecomment-794719033
@omalley can you share some message here?
This is an automated message from the Apache Git Service.
To respond to the
autumnust opened a new pull request #651:
URL: https://github.com/apache/orc/pull/651
This PR:
- Add a straightforward implementation for `Dictionary` using hash table.
- Refactored RB-Tree to make code reusable, like `VisitorContextImpl`.
Moving it into `DictionaryUtils.java` to ma
Lei Sun created ORC-757:
---
Summary: Add Hashtable implementation for dictionary
Key: ORC-757
URL: https://issues.apache.org/jira/browse/ORC-757
Project: ORC
Issue Type: Improvement
Components:
dongjoon-hyun commented on pull request #648:
URL: https://github.com/apache/orc/pull/648#issuecomment-794232710
Thank you, @wgtmac and @pgaref . Merged to master!
This is an automated message from the Apache Git Service.
To
dongjoon-hyun merged pull request #648:
URL: https://github.com/apache/orc/pull/648
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
pgaref commented on pull request #648:
URL: https://github.com/apache/orc/pull/648#issuecomment-793659019
> > > Thank you for updating, @wgtmac .
> > > Do you have other comments, @pgaref ?
> >
> >
> > Hey @dongjoon-hyun PR looks good to me as well, we just need to remove
the
wgtmac commented on pull request #648:
URL: https://github.com/apache/orc/pull/648#issuecomment-793602786
> > Thank you for updating, @wgtmac .
> > Do you have other comments, @pgaref ?
>
> Hey @dongjoon-hyun PR looks good to me as well, we just need to remove the
newly added getP
