pointearth opened a new pull request #9686:
URL: https://github.com/apache/pulsar/pull/9686


   <!--
   ### Contribution Checklist
   
   **(The sections below can be removed for hotfixes of typos)**
   -->
   ### Motivation
   When I query a topic with presto
   `presto> select * from xxxTable from 
pulsar."xxxTenant/xxxNamespace"."last-topic-value";
   
   Query 20210222_101426_00019_jvnf5, FAILED, 2 nodes
   Splits: 18 total, 0 done (0.00%)
   **0.79** [0 rows, 0B] [0 rows/s, 0B/s]
   
   You can see it cast a long time, and will not stop.
   And you can see many error messages in presto server:
   Query 20210222_101426_00019_jvnf5 failed: Duplicate key __pfn_input_topic__ 
(attempted merging values 
persistent://first-tenant-value/first-namespace-value/first-topic-value and 
persistent://second-tenant-value/second-namespace-value/second-topic-value)`
   
   **To Reproduce**
   Steps to reproduce the behavior:
   1. create 3 topics: begin-topic-value, last-topic-value, topic3
   2. create 2 pulsar functions, f1 pulls messages from first-topic-value, 
writes to second-topic-value, f2 pulls messages from second-topic-value, writes 
to last-topic-value.
   3. `presto> select * from xxxTable from 
pulsar."xxxTenant/xxxNamespace"."last-topic-value";
   4. See these errors
   
   **Expected behavior**
   query successfully, and No error.
   
   **Screenshots**
   <img width="1288" alt="Screen Shot 2021-02-24 at 00 16 09" 
src="https://user-images.githubusercontent.com/1859919/108872874-9af4a280-7635-11eb-8ce5-04c88108b12b.png";>
   
   **Desktop (please complete the following information):**
    - OS: CentOS , pulsar 2.7.0
   ### Modifications
   if key dumplicated, use new one.
   
   ### Verifying this change
   
   This change added tests and can be verified as follows:
   
     - *Added unit test in RawMessageImplTest.java*
   
   ### Does this pull request potentially affect one of the following parts:
   
   *If `yes` was chosen, please highlight the changes*
   
     - Dependencies (does it add or upgrade a dependency): (yes / **no**)
     - The public API: (yes / **no**)
     - The schema: (yes / **no** / don't know)
     - The default values of configurations: (yes / **no**)
     - The wire protocol: (yes / **no**)
     - The rest endpoints: (yes / **no**)
     - The admin cli options: (yes / **no**)
     - Anything that affects deployment: (yes / **no** / don't know)
   
   ### Documentation
   
     - Does this pull request introduces a new feature? (yes / **no**)
     - If yes, how is the feature documented? (not applicable / docs / JavaDocs 
/ not documented)
     - If a feature is not applicable for documentation, explain why? it is a 
bug
     - If a feature is not documented yet in this PR, please create a followup 
issue for adding the documentation
     - It is just like <#6390>, but it causes presto issue.
   


----------------------------------------------------------------
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:
[email protected]


Reply via email to