vogievetsky opened a new pull request #12037:
URL: https://github.com/apache/druid/pull/12037


   Right now there is a bug that can lead to parts of queries typed into the 
query view being lost:
   
   To repro:
   
   1. Run some query e.g:
   
   ```sql
   SELECT
     channel,
     COUNT(*) AS "Count"
   FROM wikipedia
   GROUP BY 1
   ORDER BY 2 DESC
   ```
   
   2. Edit the query somehow, say change "Count" to "CountBlah"
   
   3. Use the table actions to modify the query:
   
   
![image](https://user-images.githubusercontent.com/177816/145162336-0fe6fd8b-579e-49aa-978a-be1534e1c8c6.png)
   
   Notice the query edit of "Blah" is lost as the table action overwrites the 
query completely.
   
   Instead it should now provide an action (a function that takes a query and 
manipulates it) instead so the action is simply applied to the edited query.
   
   This PR also:
   - Skips the java check in the helper script to make JDK11 dev easier
   - Simplifies the format guesser
   - Adds and improves react hooks
   
   This PR has:
   - [x] been self-reviewed.
   - [x] been tested in a test Druid cluster.
   


-- 
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...@druid.apache.org

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



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

Reply via email to