rkirtir commented on code in PR #4515:
URL: https://github.com/apache/hive/pull/4515#discussion_r1275853714


##########
ql/src/test/queries/clientpositive/external_part_parquet_int_decimal.q:
##########
@@ -0,0 +1,52 @@
+dfs ${system:test.dfs.mkdir} -p 
${system:test.tmp.dir}/hive27213/ws_sold_date_sk=2451825;

Review Comment:
   Yes, you are correct! This issue is neither related with external/manged 
table nor partitions/msck .
   This is purely related to parquet file which has dictioanry defined. 
   
   Answer to your question -why parquet_int_decimal.q doesn't fail without your 
patch?
   >> Because parquet file used inside this q test does not have any 
encoding/dictionary defined inside it.
   
   If we check parquet files and converted json files for 
parquet_int_decimal_1.parquet and web_sales.parquet,  it shows attached 
dictionary in web_sales.parquet .
   
   I have changed q file names as now it is simple select on a table with 
dictionary supported parquet file
   



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

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to