phoenix-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Lars Hofhansl (JIRA)" <>
Subject [jira] [Commented] (PHOENIX-3547) Promote CATALOG.VIEW_INDEX_ID to an int
Date Fri, 18 Jan 2019 23:59:00 GMT


Lars Hofhansl commented on PHOENIX-3547:

We just came across a scenario where this breaks an old (pre-4.15) client.
The index id is now written with an int, the old client tries decode the two byte we have
currently as an int and fails.

We've seen this with local indexes and the M/R integration. This is call stack in the old
ParallelIterators(ExplainTable).appendKeyRanges(StringBuilder) line: 297    
ParallelIterators(ExplainTable).explain(String, List<String>) line: 127    
ParallelIterators(BaseResultIterators).explain(List<String>) line: 1544    
RoundRobinResultIterator.explain(List<String>) line: 153    
ScanPlan(BaseQueryPlan).getPlanSteps(ResultIterator) line: 524    
ScanPlan(BaseQueryPlan).iterator(Map<ImmutableBytesPtr,ServerCache>, ParallelScanGrouper,
Scan) line: 372    
ScanPlan(BaseQueryPlan).iterator(ParallelScanGrouper, Scan) line: 217    
ScanPlan(BaseQueryPlan).iterator(ParallelScanGrouper) line: 212    

> Promote CATALOG.VIEW_INDEX_ID to an int
> ---------------------------------------
>                 Key: PHOENIX-3547
>                 URL:
>             Project: Phoenix
>          Issue Type: New Feature
>            Reporter: Jeremy Huffman
>            Assignee: Mehdi Salarkia
>            Priority: Minor
>             Fix For: 4.15.0, 5.1.0
>         Attachments: 4.x-HBase-0.98.patch, 4.x-HBase-1.1.patch, 4.x-HBase-1.2.patch,
4.x-HBase-1.3.patch, 4.x-HBase-1.4.patch, 4.x-cdh5.11.patch, 4.x-cdh5.12.patch, 4.x-cdh5.13.patch,
4.x-cdh5.14.patch, master-PHOENIX-3547.patch
> Increase the size of CATALOG.VIEW_INDEX_ID from smallint to int to support a large number
of indexed views on a single table.
> Per James: "The code would just need to be tolerant when reading the data if the length
is two byte short versus four byte int. At write time, we'd just always write an int."
> See:

This message was sent by Atlassian JIRA

View raw message