eaugene opened a new pull request, #12244:
URL: https://github.com/apache/pinot/pull/12244

   [bugfix] 
   
   we recently encountered an error :
   ```
   java.lang.ArrayIndexOutOfBoundsException:  Index  349  out  of  bounds  for  
length  349
     at  
org.apache.pinot.segment.local.realtime.impl.forward.FixedByteMVMutableForwardIndex.getLongMV(FixedByteMVMutableForwardIndex.java:294)
     at  
org.apache.pinot.segment.spi.index.mutable.MutableForwardIndex.getLongMV(MutableForwardIndex.java:492)
     at  
org.apache.pinot.core.operator.dociditerators.MVScanDocIdIterator$LongMatcher.doesValueMatch(MVScanDocIdIterator.java:201)
     at  
org.apache.pinot.core.operator.dociditerators.MVScanDocIdIterator.applyAnd(MVScanDocIdIterator.java:104)
     at  
org.apache.pinot.core.operator.dociditerators.ScanBasedDocIdIterator.applyAnd(ScanBasedDocIdIterator.java:48)
     at  
org.apache.pinot.core.operator.docidsets.AndDocIdSet.iterator(AndDocIdSet.java:151)
     at  
org.apache.pinot.core.operator.docidsets.AndDocIdSet.iterator(AndDocIdSet.java:78)
     at  
org.apache.pinot.core.operator.docidsets.NotDocIdSet.iterator(NotDocIdSet.java:37)
     at  
org.apache.pinot.core.operator.docidsets.AndDocIdSet.iterator(AndDocIdSet.java:78)
     at  
org.apache.pinot.core.operator.DocIdSetOperator.getNextBlock(DocIdSetOperator.java:67)
     at  
org.apache.pinot.core.operator.DocIdSetOperator.getNextBlock(DocIdSetOperator.java:39)
     at  
org.apache.pinot.core.operator.BaseOperator.nextBlock(BaseOperator.java:43)
     at  
org.apache.pinot.core.operator.ProjectionOperator.getNextBlock(ProjectionOperator.java:70)
     at  
org.apache.pinot.core.operator.ProjectionOperator.getNextBlock(ProjectionOperator.java:37)
     at  
org.apache.pinot.core.operator.BaseOperator.nextBlock(BaseOperator.java:43)
     at  
org.apache.pinot.core.operator.query.GroupByOperator.getNextBlock(GroupByOperator.java:105)
     at  
org.apache.pinot.core.operator.query.GroupByOperator.getNextBlock(GroupByOperator.java:46)
     at  
org.apache.pinot.core.operator.BaseOperator.nextBlock(BaseOperator.java:43)
     at  
org.apache.pinot.core.operator.combine.GroupByCombineOperator.processSegments(GroupByCombineOperator.java:133)
     at  
org.apache.pinot.core.operator.combine.BaseCombineOperator$1.runJob(BaseCombineOperator.java:121)
     at  
org.apache.pinot.core.util.trace.TraceRunnable.run(TraceRunnable.java:40)
     at  
java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
     at  java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
     at  
java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
     at  
com.google.common.util.concurrent.TrustedListenableFutureTask$TrustedFutureInterruptibleTask.runInterruptibly(TrustedListenableFutureTask.java:125)
     at  
com.google.common.util.concurrent.InterruptibleTask.run(InterruptibleTask.java:57)
     at  
com.google.common.util.concurrent.TrustedListenableFutureTask.run(TrustedListenableFutureTask.java:78)
     at  
java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
     at  
java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
     at  java.base/java.lang.Thread.run(Thread.java:829)
   ```
   This patch fixes it by reading only till the available size in the passed 
buffer array . 
   
   I think the cause can be MutubaleDataSource._maxNumValuesPerMVEntry ( which 
is used to intialize the size of buffer array ) value updation may not be 
Atomic 


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to