Skip to content

Conversation

@huberylee
Copy link
Contributor

@huberylee huberylee commented Sep 15, 2022

Change Logs

Introducing a new action type called build to actually perform index data generation. This action contains two steps as clustering action does:

  • Generate action plan to clarify which files and which indexes need to be built;
  • Execute build index according action plan generated by step one;

Call procedure will be implemented as well to show or run build action.

Classes in package org.apache.hudi.secondary.index.lucene.hadoop were copied from package org.apache.solr.hdfs.store in Apache Solr project.

Impact

User can use call show_build(table=> '$table'[, path => $path], limit => $limit, show_involved_partition => [true/false]) to list build commits, usecall run_build(table => '$table'[, path => $path], predicate => '$predicate', show_involved_partition => [true/false]) to trigger new build action if conditions are satisfied.

Risk level: none | low | medium | high**

none

Contributor's checklist

  • Read through contributor's guide
  • Change Logs and Impact were stated clearly
  • Adequate tests were added if applicable
  • CI passed

@huberylee huberylee force-pushed the HUDI-4294-impl-build-action branch from 59e2196 to 0ce0aee Compare September 15, 2022 06:37
@huberylee
Copy link
Contributor Author

@hudi-bot run azure

@yihua yihua added priority:high Significant impact; potential bugs index big-needle-movers labels Sep 15, 2022
@huberylee huberylee force-pushed the HUDI-4294-impl-build-action branch 3 times, most recently from 146f2f9 to d018620 Compare September 20, 2022 09:46
@nsivabalan
Copy link
Contributor

@huberylee : seems interesting idea. but do we have an RFC on this already ?

@huberylee
Copy link
Contributor Author

@huberylee : seems interesting idea. but do we have an RFC on this already ?

Yes, this PR is part of the secondary index implementation. RFC is in another PR: #5370

@nsivabalan
Copy link
Contributor

👍 cool

@yihua
Copy link
Contributor

yihua commented Nov 30, 2022

Hi @huberylee , similar here, are you still actively working on this PR and resolving conflicts?

@huberylee huberylee force-pushed the HUDI-4294-impl-build-action branch from d018620 to 39a9cea Compare December 6, 2022 15:43
@huberylee
Copy link
Contributor Author

@hudi-bot run azure

@huberylee huberylee force-pushed the HUDI-4294-impl-build-action branch from 39a9cea to e55e4c0 Compare December 7, 2022 13:53
@huberylee huberylee force-pushed the HUDI-4294-impl-build-action branch 2 times, most recently from 560d192 to 88197b1 Compare December 9, 2022 14:59
@huberylee huberylee force-pushed the HUDI-4294-impl-build-action branch from 88197b1 to abc0856 Compare December 11, 2022 10:12
@hudi-bot
Copy link
Collaborator

CI report:

Bot commands @hudi-bot supports the following commands:
  • @hudi-bot run azure re-run the last Azure build

@codope
Copy link
Member

codope commented Dec 17, 2024

There is an INDEX action for this purpose now.

@codope codope closed this Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

big-needle-movers priority:high Significant impact; potential bugs release-1.0.0 size:XL PR with lines of changes > 1000

Projects

Status: 🚧 Needs Repro
Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

6 participants