You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by GitBox <gi...@apache.org> on 2020/04/22 10:50:33 UTC

[GitHub] [incubator-doris] yangzhg opened a new pull request #3381: Fix expect may produce incorrect values

yangzhg opened a new pull request #3381:
URL: https://github.com/apache/incubator-doris/pull/3381


   Fixes #3379 


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

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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


[GitHub] [incubator-doris] yangzhg commented on issue #3381: Fix expect may produce incorrect values

Posted by GitBox <gi...@apache.org>.
yangzhg commented on issue #3381:
URL: https://github.com/apache/incubator-doris/pull/3381#issuecomment-618119322


   > describe
   done


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

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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


[GitHub] [incubator-doris] kangkaisen commented on issue #3381: Fix expect may produce incorrect values

Posted by GitBox <gi...@apache.org>.
kangkaisen commented on issue #3381:
URL: https://github.com/apache/incubator-doris/pull/3381#issuecomment-618125477


   The bug cause by 
   ```
       _hash_tbl.reset(new HashTable(_child_expr_lists[0], _child_expr_lists[0], _build_tuple_size,
                                     true, _find_nulls, id(), mem_tracker(), 1024));
   ```
   should be
   ```
       _hash_tbl.reset(new HashTable(_child_expr_lists[0], _child_expr_lists[1], _build_tuple_size,
                                     true, _find_nulls, id(), mem_tracker(), 1024));
   ```


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

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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


[GitHub] [incubator-doris] kangkaisen commented on issue #3381: Fix expect may produce incorrect values

Posted by GitBox <gi...@apache.org>.
kangkaisen commented on issue #3381:
URL: https://github.com/apache/incubator-doris/pull/3381#issuecomment-617820929


   @yangzhg `open` method seems not change. which line code fix this issue. 
   
   **When you submit a issue or PR, would better describe the root cause for the issue and the core idea for your solution, thanks**
   
   
   
   


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

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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org