Github user JamesRTaylor commented on the issue:

    https://github.com/apache/phoenix/pull/239
  
    Patch looks very good, @akshita-malhotra. What's the advantage, @lhofhansl, 
of forcing users to create the snapshot themselves before starting the job? 
Wouldn't it be simpler for the snapshot to be created during the 
setup/initialization of the MR job? In the non MR case, when we want to support 
running arbitrary queries over snapshot(s), seems like we'd want Phoenix to 
create them, no? Otherwise, we'd need to provide the user with some means of 
associating a snapshot with a table name (which might get cumbersome). The 
alternative is to let Phoenix manage this transparently.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to