shivjha30 opened a new pull request, #5000:
URL: https://github.com/apache/hive/pull/5000

   ### What changes were proposed in this pull request?
   Earlier, the code flows for partitioned and non-partitioned tables were 
different. The partitioned tables skipped constraints checks before submitting 
the job for execution. This Pull request ensures that both, the partitioned and 
non-partitioned tables go through the constraints validations in 
`applyConstraintsAndGetFiles` function.
   
   
   ### Why are the changes needed?
   For partitioned tables, while executing LOAD DATA/ LOAD DATA LOCAL commands, 
the check for file existence is not executed on HiveServer2, and this in turn 
throws `java.io.FileNotFoundException` during Runtime once the job is launched. 
   This PR prevents such cases at compile time.
   
   ### Does this PR introduce _any_ user-facing change?
   No
   
   ### Is the change a dependency upgrade?
   No
   
   
   ### How was this patch tested?
   The test cases already exist. The error messages prompted back to the user 
are now consistent if the file is not found at HiveServer2
   


-- 
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: gitbox-unsubscr...@hive.apache.org

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


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

Reply via email to